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 support for semver crate #195

Closed
wants to merge 4 commits into from
Closed

add support for semver crate #195

wants to merge 4 commits into from

Conversation

OmarTawfik
Copy link
Contributor

@OmarTawfik OmarTawfik commented Dec 22, 2022

Thanks for the awesome package!

This PR adds support for the semver crate, which is a Rust-wide standard now for reading/writing/comparing/validating versions. It also has its own serde feature for serializing/deserializing Version types into JSON strings.

Any comments/feedback is really appreciated. Would love to merge this and integrate it into our own projects!

@OmarTawfik
Copy link
Contributor Author

cc @GREsau

@OmarTawfik OmarTawfik changed the title add support for semver package add support for semver crate Dec 26, 2022
@OmarTawfik
Copy link
Contributor Author

Added a Regex pattern for the resulting string type, which is a permissive version of the official spec at: https://semver.org/

@OmarTawfik
Copy link
Contributor Author

@GREsau I wonder if you have time to take a look.
Is there someone else perhaps that is maintaining this crate?
Thanks!

@OmarTawfik
Copy link
Contributor Author

It seems like the project is still active, with new updates this week. Would love to see this shipped!

@GREsau Thanks for all the hard work maintaining this incredibly useful crate!

@sunshowers
Copy link

Can this use the official validation regex? Doesn't seem like there's a particular reason not to.

@OmarTawfik
Copy link
Contributor Author

@sunshowers you are absolutely correct. updated!

@jayvdb
Copy link

jayvdb commented Jun 28, 2023

This is largely the same as #182

@OmarTawfik
Copy link
Contributor Author

@jayvdb How so?

@jayvdb
Copy link

jayvdb commented Jun 29, 2023

oh sorry wrong one - I meant #74

This was referenced Aug 27, 2023
@GREsau
Copy link
Owner

GREsau commented Aug 27, 2023

Merged in #238, better late than never right? 😅

Thanks for the PR

@GREsau GREsau closed this Aug 27, 2023
@OmarTawfik
Copy link
Contributor Author

Thank you!

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.

4 participants