-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
tidy-viewer 1.4.6 #106452
tidy-viewer 1.4.6 #106452
Conversation
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.
Not sure why the checksum doesn't match, but here's the right one.
@carlocab I made the mistake of running |
package update Update tidy-viewer.rb
e113753
to
2cab33b
Compare
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.
Thanks! For future reference, our preferred commit message style for version bumps is something like
tidy-viewer 1.4.6
Additionally, please don't push merge commits to your PR branch. These it more difficult to merge your PR because of our repo's linear history requirement. (That's where the automerge-skip
label came from.)
Finally, if you have a local installation of brew
handy, you can automate all this with something like
brew bump-formula-pr --version=1.4.6 tidy-viewer
Or, failing that, you can use this GitHub Action to automate the creation of version bump PRs when you cut a new release.
That works. is the command |
You can run it anywhere. Note that |
🤖 A scheduled task has triggered a merge. |
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?