Skip to content

Fix what the last audit of 2.0 found - #274

Merged
lyrixx merged 1 commit into
mainfrom
audit-fixes
Nov 10, 2026
Merged

lyrixx merged 1 commit into
mainfrom
audit-fixes

Conversation

@lyrixx

@lyrixx lyrixx commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

The fixes of the last audit before 2.0.0. 1.7.2 already has the ones that apply to 1.x.

What changes for users, all in the changelog and the upgrade guide:

  • Parser\DiffParser and Parser\ParserBase are internal, and GitExceptionInterface extends \Throwable: free now, a major version after the tag.
  • CliDriver runs git with LANGUAGE=C. A translated git added a branch to getIncludingBranches() and hid the error line of a ProcessException.
  • LibGit2Driver refuses a checkout or a merge that removes a checked out submodule, where it deleted its directory, repository included.

CI now runs the suite on macOS and Windows too. PHP nightly may fail without failing the workflow.

Before the tag freezes the API: the parsers are internal, and
GitExceptionInterface extends Throwable.
@lyrixx
lyrixx merged commit fdba77f into main Nov 10, 2026
18 checks passed
@lyrixx
lyrixx deleted the audit-fixes branch November 10, 2026 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant