shnewto / bnf

Parse BNF grammar definitions
MIT License
256 stars 22 forks source link

New release process (release with a manually triggered action) #139

Closed shnewto closed 9 months ago

shnewto commented 9 months ago

This PR adds an action that can be manually triggered to handle bumping the Cargo.toml version, adding a corresponding tag to the repo, and publishing to crates.io. The full proposed process is documented in the RELEASE.md that's been added here too.

Some things I thought about then decided against before settling on this solution:

(There are also a couple fixes for clippy lints that made it in here and some fixes to other errors that showed up in CI)

coveralls commented 9 months ago

Coverage Status

coverage: 96.67% (+0.08%) from 96.589% when pulling 6c2a92554bd652c26581a09b827b521d2700a1e4 on publish-crate-action into 465b7977ca63f81c818670250ec4d8968bb0e425 on main.

shnewto commented 9 months ago

Oh something maybe notable in the implementation you might catch: runs-on: macos-latest. This is something I've adopted as a default instead of ubuntu-latest in other crates because I don't have as easy access to an ubuntu machine to troubleshoot issues, and macos-latest also seems to less often need preliminary sudo apt ... steps to pull in extra system dependencies.