Skip to content

[Contributing.md] Suggestion to improve the contribution best practices #9928

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

Merged
merged 2 commits into from
Feb 16, 2024

Conversation

cwilso
Copy link
Contributor

@cwilso cwilso commented Feb 9, 2024

This is merely a suggestion to improve the contribution best practices to reflect (what I think is) the actual status quo.

This is based on an attempt to figure out how we could have avoided some incidents that ruffled a few feathers, by being clearer about expectations; I'd welcome feedback about whether this is useful or if it could be improved in some other way.

@cwilso cwilso requested review from frivoal and fantasai February 9, 2024 01:11
Addressing @fantasai's feedback and removing double-whitespace-after-period.
@cwilso cwilso requested a review from fantasai February 12, 2024 17:18
Copy link
Member

@hober hober left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@cwilso
Copy link
Contributor Author

cwilso commented Feb 12, 2024

As this isn't a spec, unfortunately these guidelines aren't quite applicable (since there's no Editor :P ). As I note a couple of approvals and a thumbs-up from one of the chairs, I'll leave it to you all to decide when it's appropriate to merge once @fantasai has given it a re-review.

Copy link
Member

@tantek tantek left a comment

Choose a reason for hiding this comment

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

LGTM, this is consistent with my rough understanding of current WG expectations.

@cwilso cwilso marked this pull request as ready for review February 15, 2024 17:45
@astearns astearns merged commit 207e931 into w3c:main Feb 16, 2024
@astearns
Copy link
Member

We will announce this update on the Feb 28 call and can take further improvements

@bramus
Copy link
Contributor

bramus commented Feb 28, 2024

To those following: filed a small PR to clarify where PRs should originate from: #10009

@frivoal frivoal removed the Agenda+ label Jun 12, 2024
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.

7 participants