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

PEP 752: Address feedback, round 2 #3922

Merged
merged 1 commit into from
Aug 24, 2024
Merged

Conversation

ofek
Copy link
Sponsor Contributor

@ofek ofek commented Aug 24, 2024

  1. Changed the Delegate from Donald Stufft to Dustin Ingram.
  2. Renamed private/public terminology to private/shared.
  3. Allowed community projects the ability to have private namespaces but still should be shared-only. The determination is at the discretion of reviewers.
  4. Added a separate visual indicator for projects that are part of a shared namespace and are not owned by a grant holder.
  5. Added a separate visual indicator for projects matching a private namespace that existed before the grant.
  6. Made it explicit that root grants cannot overlap e.g. if there is a grant for foo-bar then one cannot apply for foo.
  7. Adjusted language about the application review prioritization of corporate organizations.
  8. Clarified that it is forbidden to replicate namespace configuration from other repositories.
  9. Clarified the link between name-squatting attacks and dependency confusion.
  10. Require a page in the UI for each namespace with metadata like the current owners.
  11. Rather than encouraging a page that enumerates every namespace, this is now expressly forbidden.
  12. Added a recommendations section that lives outside of the concrete specification.

📚 Documentation preview 📚: https://pep-previews--3922.org.readthedocs.build/

@ofek ofek requested a review from warsaw as a code owner August 24, 2024 19:38
@warsaw warsaw merged commit 82f5112 into python:main Aug 24, 2024
6 checks passed
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