Skip to content

Python Versioning #20

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

Closed

Conversation

Allstreamer
Copy link
Collaborator

@Allstreamer Allstreamer commented Aug 9, 2022

Implements Equality and Ordering Operations for Python Pep-440 Versions as described in the image below:


www debuggex com_

Intergrates very nicely with @Kiwifuit's #18 Where His version requirement code can be supplemented by my system

- Implements PackageVersion struct which allows parsing of a python pep-440 version string
- Added package_version to main
@Allstreamer Allstreamer marked this pull request as draft August 9, 2022 21:55
- Added PostHead & PostHeader
- Finished Post Version Parsing
- Added More tests
- Added DevHead
- Finished Dev Version processing
- Realized that local version can be +<anything>.<anything>.<anything> and so on
- Ran cargo fmt
- Added v1.1 unit test
- Made negative unit test fail more spesific
- Implemented equality
- Added equality unit-tests
- Changed DevHead to struct from enum
- Added Ordering for Dev
- Finished Post Ordering
- Added Post Ordering Unit tests
- Ran cargo fmt
- Finished Pre Version Ordering
- Added Pre Version unit tests
- Renamed VersionRelease to ReleaseHeader
- Added ReleaseHeader unit tests
- Finished All Ordering
- Added PackageVersion doc-strings
- Made ReleaseHeader Unit test more verbose
- Added doc strings
- Added usage example
@Allstreamer Allstreamer marked this pull request as ready for review August 10, 2022 16:27
@Kiwifuit
Copy link

Kiwifuit commented Aug 14, 2022

@Allstreamer in lines 133 and 134, where are Derivative and derivative from respectively?

Here's a link to the lines in question

@Allstreamer
Copy link
Collaborator Author

Ah misread the question at first

@Kiwifuit
Copy link

My IDE cannot resolve lines 133 and 134 in your code for some reason

@Allstreamer
Copy link
Collaborator Author

Check main.rs & Cargo.toml. There their usage is denfined

@Allstreamer
Copy link
Collaborator Author

You forgot to include all changes In your branch

Allstreamer added a commit that referenced this pull request Aug 15, 2022
- Moved changes from #20 to 25-add-python-version-parsing-pep-440 branch
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.

2 participants