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

Update DEP 7 terminology to match current landscape #91

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

thibaudcolas
Copy link
Member

@thibaudcolas thibaudcolas commented Sep 10, 2024

This is a follow-up to @jacobian’s #81, updating DEP 7 so its terminology matches current operations.

As of today I believe the relevant projects are:

This policy also came up as part of Django ticket #33620 Accessibility in pipeline.

Copy link
Member

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

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

This looks right. Thanks @thibaudcolas

@thibaudcolas
Copy link
Member Author

👋 I’m not sure if this needs further review, or someone with appropriate permissions to merge?

@jacobian
Copy link
Member

@django/steering-council - any feedback? If not, I say yeah let's merge in about a week.

@thibaudcolas
Copy link
Member Author

I’ve also opened #95 as a follow-up with even more terminology updates. The only one I’ve left out for now is DEP 10.

Copy link
Member

@adamchainz adamchainz left a comment

Choose a reason for hiding this comment

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

One small comment, otherwise LGTM

final/0007-official-projects.rst Outdated Show resolved Hide resolved
@thibaudcolas
Copy link
Member Author

Looks like this is now ready to merge if anyone with write access is around?

@ubernostrum
Copy link
Member

Perhaps this should wait until after the SC election so they can properly vote on whether to accept the changes?

@thibaudcolas
Copy link
Member Author

@ubernostrum I was under the impression the changes here would fall in the "housekeeping" bucket and wouldn’t require a vote, per past advice on similar changes (Updates to DEPs 1, 10, and 12 to reflect current governance).

@nanuxbe
Copy link
Contributor

nanuxbe commented Jan 6, 2025

+1

Copy link
Contributor

@LilyFoote LilyFoote left a comment

Choose a reason for hiding this comment

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

I think line 157 needs updating. It currently has:

Once a team is assembled, the project will be taken for full discussion on
mailing lists and other archived public fora...

and this should probably mention primarily the Django Forum now.

final/0007-official-projects.rst Outdated Show resolved Hide resolved
@frankwiles
Copy link
Member

+1

Co-authored-by: Adam Johnson <[email protected]>
Co-authored-by: Lily Foote <[email protected]>
Co-authored-by: Tim Schilling <[email protected]>
@tim-schilling
Copy link
Member

Squashed the commits and merging.

@tim-schilling tim-schilling merged commit ec95d26 into django:main Jan 14, 2025
@tim-schilling
Copy link
Member

Thanks everyone!

@thibaudcolas thibaudcolas deleted the dep-7-freshness branch January 23, 2025 01:56
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.

9 participants