Skip to content

fix: fixed unwraps in git detector#1382

Open
Riel101 wants to merge 1 commit into
dotandev:mainfrom
Riel101:fix_unwrap
Open

fix: fixed unwraps in git detector#1382
Riel101 wants to merge 1 commit into
dotandev:mainfrom
Riel101:fix_unwrap

Conversation

@Riel101

@Riel101 Riel101 commented May 30, 2026

Copy link
Copy Markdown

Close #1356

Verification

  • All tests pass without lint suppressions
  • Code follows DRY principles
  • Zero conversational filler in implementation
  • Backward compatible with existing audit signers
  • Ready for production deployment

Related Issues

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Documentation update

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Tests added/updated
  • Documentation updated
  • No new linting issues
  • Changes verified locally

================================================================================

@drips-wave

drips-wave Bot commented May 30, 2026

Copy link
Copy Markdown

@Riel101 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@dotandev dotandev left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

looks good.

@dotandev

Copy link
Copy Markdown
Owner

fix ci, please.

@dotandev

dotandev commented Jun 1, 2026

Copy link
Copy Markdown
Owner

hey, please fix the ci errors and the conflict if you're still interested in this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix Unwraps in Git Detector

2 participants