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

Draft: Defining clear rules regarding merging a PR #678

Merged
merged 7 commits into from
Sep 6, 2023
Merged

Conversation

Noc2
Copy link
Collaborator

@Noc2 Noc2 commented Jul 13, 2023

Given that we now have separated the website repo and the spec repo, it might make sense to define clear rules regarding who decided what should be merged and what shouldn't. Therefore I quickly came up with a Spec Committee and potentially we can also integrate some additional rules (e.g., the spec needs to be based on research and/or existing implementations). My main goals here are:

  • convince more implementers to actively participate when it comes to the specification
  • overall improve the quality
  • an initial step to further decentralize the spec

Noc2 added 2 commits July 13, 2023 10:58
Initial clean up
Draft Spec Committee
Copy link
Contributor

@0xCaso 0xCaso left a comment

Choose a reason for hiding this comment

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

lgtm!

README.md Outdated Show resolved Hide resolved
Noc2 and others added 4 commits July 13, 2023 16:55
Co-authored-by: caso <matteocasonato@protonmail.com>
Update Committee
Add Kamil
Quadrivium instead of Soramitsu
@Noc2 Noc2 marked this pull request as ready for review August 2, 2023 15:20
@Noc2 Noc2 requested review from bhargavbh and heversonbr August 2, 2023 15:20
@danforbes
Copy link

Per my comment here, it seems like this may be a good opportunity to deploy the Polkadot Fellowship.

@Noc2
Copy link
Collaborator Author

Noc2 commented Aug 7, 2023

The idea behind introducing the Spec Committee, in addition to the technical fellowship:

  • was to have a group with a clearly defined responsibility to maintain the spec
  • and to convince more people to contribute

That said, I'm happy also to move the spec to the fellowship and to discuss or help fellowship members with the transition.

@Noc2
Copy link
Collaborator Author

Noc2 commented Sep 4, 2023

Any additional comments here? Given the response here, I would like to at least give it a try and see if this leads to more/better contributions to the spec.

Copy link
Contributor

@bhargavbh bhargavbh left a comment

Choose a reason for hiding this comment

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

LGTM, we need to add non-W3F members of the committee as outside collaborators to the repo

Copy link
Contributor

@heversonbr heversonbr left a comment

Choose a reason for hiding this comment

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

LGTM

Add myself, as requested
@Noc2 Noc2 merged commit f5f00d4 into w3f:main Sep 6, 2023
1 check 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.

6 participants