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

Pyproject #1203

Merged
merged 2 commits into from
Nov 28, 2024
Merged

Pyproject #1203

merged 2 commits into from
Nov 28, 2024

Conversation

mdellweg
Copy link
Member

No description provided.

description = "pulp-deb plugin for the Pulp Project"
readme = "README.md"
authors = [
{name="Matthias Dellweg", email="[email protected]"},
Copy link
Collaborator

Choose a reason for hiding this comment

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

@mdellweg Since this email presumably no longer exists, do you want to update this section? Should we add more "authors", or is there a separate field for current "maintainers"?

Copy link
Member Author

Choose a reason for hiding this comment

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

Right. I wanted to ask you for suggestions.
Obviously, authors is a list, so we can add more.
But i think you can also add "maintainers".

https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#authors-maintainers

How about we merge this as the very transition it is, and you can adjust the fields afterwards?

Copy link
Contributor

Choose a reason for hiding this comment

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

How about we merge this as the very transition it is, and you can adjust the fields afterwards?

This sounds good to me. It appears that it didn't bother anyone so far that the email is not used, though I can't of course not be sure, maybe once we add more we suddenly get an influx of request 🙃

@mdellweg mdellweg merged commit 30a73a0 into pulp:main Nov 28, 2024
12 checks passed
@mdellweg mdellweg deleted the pyproject branch November 28, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants