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 local sections #8

Merged
merged 6 commits into from
Dec 3, 2020

Conversation

mariusvniekerk
Copy link
Contributor

@mariusvniekerk mariusvniekerk commented Dec 1, 2020

This will correctly deal with pep440 version strings of the form X.Y+local. (as per https://www.python.org/dev/peps/pep-0440/#local-version-identifiers)

The plan is to add the additional support into tools that make use of this such as poetry-dynamic-versioning

This will correctly deal with pep440 version strings of the form X.Y+local
Copy link
Owner

@mtkennerly mtkennerly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I just have a few comments/requests, but I think this is a good addition to the project.

dunamai/__init__.py Outdated Show resolved Hide resolved
dunamai/__init__.py Outdated Show resolved Hide resolved
dunamai/__init__.py Outdated Show resolved Hide resolved
dunamai/__init__.py Outdated Show resolved Hide resolved
@mariusvniekerk
Copy link
Contributor Author

@mtkennerly made those requested changes

Copy link
Owner

@mtkennerly mtkennerly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great :D

) -> str:
"""
Create a string from the version info.

:param metadata: Metadata (commit, dirty) is normally included in
the local version part if post or dev are set. Set this to True to
the tagged_metadata version part if post or dev are set. Set this to True to
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like an accidental find/replace 🙃 But I need to fix up the reference to post/dev anyway, so I can take care of it.

@mtkennerly mtkennerly merged commit 7825eaf into mtkennerly:master Dec 3, 2020
@mtkennerly mtkennerly added this to the v1.5.0 milestone Dec 3, 2020
@mtkennerly
Copy link
Owner

@mariusvniekerk This is now available in v1.5.0 🎉

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