Skip to content

waive task criteria with aviator dismiss - #55

Merged
aviator-app[bot] merged 1 commit into
masterfrom
waive-criteria
Oct 29, 2026
Merged

aviator-app[bot] merged 1 commit into
masterfrom
waive-criteria

Conversation

@tulioz

@tulioz tulioz commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

dismissals now take action: "waive" on a stable_key entry, so an AC verify can't pass can be waived instead of deleted

@tulioz
tulioz requested a review from a team as a code owner October 29, 2026 19:25
@aviator-app

aviator-app Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator (commit f16bcb6).


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@aviator-app

aviator-app Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

✅ FlexReview Status

Common Owner: aviator-co/engineering (expert-load-balance assignment)
Owner and Assignment:

  • ✅ aviator-co/engineering (expert-load-balance assignment)
    Owned Files
    • 🔒 CLAUDE.md
    • 🔒 README.md
    • 🔒 cmd/aviator/dismiss.go
    • 🔒 cmd/aviator/dismiss_test.go
    • 🔒 internal/api/verify_dismiss.go

Review SLO: 7 business hours if PR size is <= 200 LOC for the first response.

@aviator-app
aviator-app Bot requested a review from brain-crystal October 29, 2026 19:25
@aviator-app
aviator-app Bot merged commit f16bcb6 into master Oct 29, 2026
4 checks passed
@aviator-app
aviator-app Bot deleted the waive-criteria branch October 29, 2026 22:00
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.

2 participants