Skip to content

Comments

Improve update check with semver#891

Merged
GyulyVGC merged 9 commits intoGyulyVGC:mainfrom
aguacero7:improve-versionning
Sep 18, 2025
Merged

Improve update check with semver#891
GyulyVGC merged 9 commits intoGyulyVGC:mainfrom
aguacero7:improve-versionning

Conversation

@aguacero7
Copy link
Contributor

No description provided.

@GyulyVGC GyulyVGC added the enhancement New feature, request, or improvement label Jul 22, 2025
@GyulyVGC GyulyVGC added this to the v1.4.1 milestone Jul 22, 2025
@GyulyVGC
Copy link
Owner

Thanks, awesome!

It'd be amazing if you could add some tests to assert that it behaves as expected trying with different version numbers

@GyulyVGC
Copy link
Owner

Nevermind I read the specs at https://docs.rs/semver/1.0.26/semver/struct.Version.html#total-ordering

And given that the library is maintained by dtolnay, I trust it 😅

@aguacero7
Copy link
Contributor Author

Hahahah, I agree i should have added some but i mean semver seems very reliable.
However, I'll fix the indentation and formatting that's currently breaking the CI.

@GyulyVGC GyulyVGC modified the milestones: v1.4.1, v1.4.2 Aug 30, 2025
@aguacero7
Copy link
Contributor Author

Oops, completely forgot about that

@aguacero7
Copy link
Contributor Author

That's it !

@aguacero7
Copy link
Contributor Author

Finally ahah

@GyulyVGC
Copy link
Owner

GyulyVGC commented Sep 7, 2025

Thanks!

@GyulyVGC
Copy link
Owner

@all-contributors please add @aguacero7 for code.

@allcontributors
Copy link
Contributor

@GyulyVGC

I've put up a pull request to add @aguacero7! 🎉

@GyulyVGC GyulyVGC merged commit 69224fb into GyulyVGC:main Sep 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, request, or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants