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

Remove unused setuptools remainings #13684

Merged
merged 2 commits into from
Mar 12, 2024
Merged

Remove unused setuptools remainings #13684

merged 2 commits into from
Mar 12, 2024

Conversation

dbast
Copy link
Member

@dbast dbast commented Mar 12, 2024

conda once was once packaged by setuptools + wheel extension.

#12509 did the switch to hatch, which has its own wheel builder: https://github.com/pypa/hatch/blob/master/backend/src/hatchling/builders/wheel.py#L160

Thus the wheel package (containing the setuptools wheel extension) and the MANIFEST.in file (describes what setuptools should include) can be removed. The hatch file selection is not based on the MANIFEST.in, see https://hatch.pypa.io/1.9/plugins/builder/wheel/#default-file-selection

Description

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

conda once was once packaged by setuptools + wheel extension.

conda#12509 did the switch to hatch, which has its own wheel builder:
https://github.com/pypa/hatch/blob/master/backend/src/hatchling/builders/wheel.py#L160

Thus the wheel package (containing the setuptools wheel extension) and the
MANIFEST.in file (describes what setuptools should include) can be removed.
The hatch file selection is not based on the MANIFEST.in, see
https://hatch.pypa.io/1.9/plugins/builder/wheel/#default-file-selection
@dbast dbast requested a review from a team as a code owner March 12, 2024 14:18
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 12, 2024
Copy link
Contributor

@dholth dholth 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 getting rid of MANIFEST.in :)

Copy link

codspeed-hq bot commented Mar 12, 2024

CodSpeed Performance Report

Merging #13684 will not alter performance

Comparing dbast:str (0f33a72) with main (beccf64)

Summary

✅ 21 untouched benchmarks

Copy link
Member

@jezdez jezdez left a comment

Choose a reason for hiding this comment

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

Pouring one out to good ol' manifest templates.

@jezdez jezdez enabled auto-merge (squash) March 12, 2024 15:13
@jezdez jezdez merged commit 1f73fba into conda:main Mar 12, 2024
66 checks passed
dholth pushed a commit that referenced this pull request Mar 20, 2024
jezdez pushed a commit to Callek/conda that referenced this pull request Apr 9, 2024
beeankha added a commit to beeankha/conda that referenced this pull request May 3, 2024
beeankha added a commit to beeankha/conda that referenced this pull request May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants