Skip to content
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

Add semver check to CI #1005

Closed
wants to merge 2 commits into from
Closed

Conversation

tcharding
Copy link
Member

Draft because I currently cannot get cargo semver to run locally so I'm going to abuse CI to run it.

Add a CI job to check if we have correctly set the crate version based on Semantic Versioning rules.

This job was copied from: libp2p/rust-libp2p#2647

And uses this tool: https://github.com/rust-lang/rust-semverver

Thanks to @thomaseizinger for bringing this to my attention.

We have increased the MSRV, this requires a 'minor' version bump (since
we are pre v1.0).
Add a CI job to check if we have correctly set the crate version based
on Semantic Versioning rules.

This job was copied from: libp2p/rust-libp2p#2647

And uses this tool: https://github.com/rust-lang/rust-semverver
@thomaseizinger
Copy link
Contributor

Lol, internal compiler error 🙃

@tcharding
Copy link
Member Author

Damn, I guess I'll have to make this work locally ... another day.

@tcharding tcharding closed this May 25, 2023
@tcharding tcharding deleted the 05-25-semver branch May 25, 2023 03:42
@tcharding
Copy link
Member Author

One year has passed since I started this and now the tool is deprecated :(

@thomaseizinger
Copy link
Contributor

One year has passed since I started this and now the tool is deprecated :(

The new kid on the block is cargo semver-checks.

@tcharding
Copy link
Member Author

Thanks man!

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.

None yet

2 participants