Skip to content

Conversation

@tidy-dev
Copy link
Contributor

Introduced a new GitError type 'PushWithSecretDetected' and updated the GitErrorRegexes mapping to handle GitHub push protection errors. Added a test case to verify parsing of this specific error scenario.

Introduced a new GitError type 'PushWithSecretDetected' and updated the GitErrorRegexes mapping to handle GitHub push protection errors. Added a test case to verify parsing of this specific error scenario.
@tidy-dev tidy-dev requested a review from sergiou87 April 21, 2025 14:33
Copy link
Member

@sergiou87 sergiou87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically this is not a git error and therefore it should live in Desktop… not sure how much we should care about it if we eventually move this into the Desktop repo, though.

@tidy-dev tidy-dev merged commit 12395cb into main Apr 21, 2025
6 checks passed
@tidy-dev tidy-dev deleted the Detect-push-blocked-with-secret branch April 21, 2025 14:43
@tidy-dev tidy-dev mentioned this pull request Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants