Skip to content

MNT: Re-add pyproject.toml #1133

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

Merged
merged 3 commits into from
Sep 7, 2022
Merged

MNT: Re-add pyproject.toml #1133

merged 3 commits into from
Sep 7, 2022

Conversation

effigies
Copy link
Member

@effigies effigies commented Sep 4, 2022

Time to try using more modern build metadata again.

  • Use pyproject.toml to mark build-time dependencies , which also allows us to move to the non-vendored flavor of versioneer
  • Move most package metadata out of setup.cfg. I don't really want to rewrite the extras interpolation, so I'm marking that dynamic for now.

@matthew-brett I've verified that pip install --user -e . works with recent setuptools, which was the issue in #859 that made us drop this before.

@effigies effigies force-pushed the mnt/pyproject branch 3 times, most recently from af422e4 to a7cd030 Compare September 4, 2022 18:01
@effigies effigies changed the title MNT: Re-add pyproject.toml, unvendor versioneer MNT: Re-add pyproject.toml Sep 4, 2022
@codecov
Copy link

codecov bot commented Sep 4, 2022

Codecov Report

Merging #1133 (7ee6327) into master (16bdd14) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1133   +/-   ##
=======================================
  Coverage   91.80%   91.80%           
=======================================
  Files         101      101           
  Lines       12445    12445           
  Branches     2430     2430           
=======================================
  Hits        11425    11425           
  Misses        693      693           
  Partials      327      327           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@matthew-brett matthew-brett left a comment

Choose a reason for hiding this comment

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

Looks good to me. Are you still iterating?

@effigies
Copy link
Member Author

effigies commented Sep 7, 2022

Nope, thanks for the review!

@effigies effigies merged commit a0bbc97 into nipy:master Sep 7, 2022
@effigies effigies deleted the mnt/pyproject branch December 27, 2022 19:08
@effigies effigies added this to the 5.0.0 milestone Jan 3, 2023
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