Skip to content

Build source packages with PEP 625 compliant filenames #285

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 10, 2025
Merged

Build source packages with PEP 625 compliant filenames #285

merged 1 commit into from
Feb 10, 2025

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Feb 10, 2025

We bump setuptools (and related dependencies) so the built source packages have PEP 625 compliant filenames. This is required by PyPI:

In the future, PyPI will require all newly uploaded source distribution filenames to comply with PEP 625. Any source distributions already uploaded will remain in place as-is and do not need to be updated.

Specifically, your recent upload of 'frequenz-api-common-0.6.4.tar.gz' is incompatible with PEP 625 because it does not contain the normalized project name 'frequenz_api_common'.

@llucax
Build source packages with PEP 625 compliant filenames
b6c21dd 
We bump setuptools (and related dependencies) so the built source
packages have PEP 625 compliant filenames. This is required by PyPI:

> In the future, PyPI will require all newly uploaded source
> distribution filenames to comply with PEP 625. Any source
> distributions already uploaded will remain in place as-is and do not
> need to be updated.
>
> Specifically, your recent upload of 'frequenz-api-common-0.6.4.tar.gz'
> is incompatible with PEP 625 because it does not contain the
> normalized project name 'frequenz_api_common'.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax requested a review from a team as a code owner February 10, 2025 11:10
@llucax llucax requested review from matthias-wende-frequenz and removed request for a team February 10, 2025 11:10
@llucax llucax self-assigned this Feb 10, 2025
@llucax llucax added this to the v0.6.5 milestone Feb 10, 2025
@github-actions github-actions bot added the part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) label Feb 10, 2025
@llucax llucax added cmd:skip-release-notes It is not necessary to update release notes for this PR part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:tech-debt Improves the project without visible changes for users and removed part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) labels Feb 10, 2025
@llucax llucax enabled auto-merge February 10, 2025 11:11
@llucax llucax merged commit acc3401 into frequenz-floss:v0.6.x Feb 10, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tiyash-basu-frequenz tiyash-basu-frequenz tiyash-basu-frequenz approved these changes

@matthias-wende-frequenz matthias-wende-frequenz Awaiting requested review from matthias-wende-frequenz matthias-wende-frequenz is a code owner automatically assigned from frequenz-floss/api-team

Assignees

@llucax llucax

Labels

cmd:skip-release-notes It is not necessary to update release notes for this PR part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:tech-debt Improves the project without visible changes for users

Projects

None yet

Milestone

v0.6.5

Development

Successfully merging this pull request may close these issues.

2 participants

@llucax @tiyash-basu-frequenz