New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
formula_auditor: split audit_revision_and_version_scheme
.
#16466
formula_auditor: split audit_revision_and_version_scheme
.
#16466
Conversation
This allows passing --skip-revision-audit to brew test-bot. Depends on Homebrew/brew#16466 Depends on Homebrew/homebrew-test-bot#998
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll ✅ to ensure we can deploy a fix, but I think the correct thing to do here is to make the revision
audit smarter.
|
||
current_version_scheme = formula.version_scheme | ||
|
||
previous_committed, = committed_version_info |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#committed_version_info
is cached, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, looks like there are callers that need fixing? |
Separate this into two methods so we can have separate skips for each.
4385d4e
to
724e14e
Compare
I'm game for that but I think it's a longer lift and this code is really hard to get right. I think having more labels that allow maintainers to override some of these audits when they don't make sense makes the most sense (at least in the short-term). |
@carlocab Thanks, should be fixed. |
Generally ok with skipping But in general 👍 |
@Bo98 Yeh, I'd be game to split this audit (and others) to be even more granular as desired. |
Separate this into two methods so we can have separate skips for each.