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

setup.py -> pyproject.toml #8

Merged
merged 4 commits into from
Apr 25, 2024
Merged

Conversation

philipmnel
Copy link
Contributor

Todos

Notable points that this PR has either accomplished or will accomplish.

  • Replace setup.py with pyproject.toml
  • Replace versioneer with setuptools-git-versioning, which does similar thing without a bunch of extra files laying around

Status

  • Ready to go

Copy link
Collaborator

@loriab loriab left a comment

Choose a reason for hiding this comment

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

If you have confidence in this versioning setup, lgtm!

pyproject.toml Outdated Show resolved Hide resolved
Co-authored-by: Lori A. Burns <[email protected]>
Copy link
Collaborator

@loriab loriab 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 modernization!

@loriab loriab merged commit c819080 into carlosborca:master Apr 25, 2024
2 checks passed
@philipmnel
Copy link
Contributor Author

Yeah, the version ends up looking something like crystalatte-0.1.191+git.3b65f4fe.dirty.
191 means 191 commits since last tag, so it would be good to do a 0.2 sometime

@philipmnel philipmnel deleted the pyproject branch April 25, 2024 20:39
@loriab
Copy link
Collaborator

loriab commented Apr 25, 2024

good to do a 0.2 sometime

agreed. I've neither blocks nor urgency on a tag. you?

@philipmnel
Copy link
Contributor Author

My only preference is that there would be tag that reproduces current results. But if there's any outstanding non-breaking changes needed for QCA stuff, its fine to wait.

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