Skip to content

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 fantasai and frivoal 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
Contributor

@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