Skip to content

Add editor assignments, permission scheme, and substantive changes - #95

Merged
justinwb merged 20 commits into
masterfrom
add-editor-assignments
Aug 28, 2019
Merged

Add editor assignments, permission scheme, and substantive changes#95
justinwb merged 20 commits into
masterfrom
add-editor-assignments

Conversation

@justinwb

@justinwb justinwb commented Aug 3, 2019

Copy link
Copy Markdown
Member

This pull request aims to address the concerns raised across the issues and pull requests itemized below. Of particular note, it merges the branches in pulls #82 and #78, and discards #81, which was no longer necessary.

The changes proposed are the result of collaboration with many of the people involved in the issues and pulls listed. Most importantly, the issues have been discussed with @timbl directly, and the proposed approach related to editor assignments and the editorial team is his recommended solution.

@RubenVerborgh

Copy link
Copy Markdown
Contributor

👍

@kjetilk kjetilk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have written two comments inline.

Comment thread README.md Outdated
Comment thread README.md Outdated
@acoburn

acoburn commented Aug 6, 2019

Copy link
Copy Markdown
Member

👍

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
@kjetilk
kjetilk self-requested a review August 8, 2019 19:40

@kjetilk kjetilk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good! I'm 👍 I'm a bit unsure how large the editorial team will be an if it might grow out of proportion, but lets just see.

@Mitzi-Laszlo

Copy link
Copy Markdown

What would be great to include as well is:

  • the aim/ scope of the specification
  • the aim/ scope of the roadmap
  • the aim/ scope of the documentation
  • the respective repositories to which proposals in the form of pull requests to the specification, roadmap, and documentation should be made
  • the aim of each repo and how it connects to this process

@justinwb

justinwb commented Aug 9, 2019

Copy link
Copy Markdown
Member Author

What would be great to include as well is:

  • the aim/ scope of the specification
  • the aim/ scope of the roadmap
  • the aim/ scope of the documentation
  • the respective repositories to which proposals in the form of pull requests to the specification, roadmap, and documentation should be made
  • the aim of each repo and how it connects to this process

@Mitzi-Laszlo Agree there's value in providing more context around these. Have created a new issue that we can use to track them: #104

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread editors.md Outdated
@Mitzi-Laszlo

Copy link
Copy Markdown

Who will be responsible for moderating the implementation of this process?

@RubenVerborgh

Copy link
Copy Markdown
Contributor

The community. It’s self-governing.

@justinwb
justinwb requested a review from RubenVerborgh August 17, 2019 03:05

@RubenVerborgh RubenVerborgh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks a lot, Justin!

Comment thread README.md Outdated

@csarven csarven left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Mentioned df05c4c#r316179594 .. leaving the response and possible action to your discretion.

@timbl timbl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Two suggestions not crucial

Comment thread README.md Outdated
Comment thread README.md
@justinwb
justinwb merged commit 977578c into master Aug 28, 2019
@RubenVerborgh

Copy link
Copy Markdown
Contributor

Thank you @justinwb for getting this through 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

process proposal Process proposal to be reviewed by Solid Director

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants