-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
[RFC 0138] Developing RFCs in repositories #138
Changes from 3 commits
9cf87fb
c0a80fb
e6a921c
94bad5e
08e64fc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,94 @@ | ||
--- | ||
feature: rfc-repos | ||
start-date: 2022-01-10 | ||
author: Silvan Mosberger | ||
co-authors: (find a buddy later to help out with the RFC) | ||
shepherd-team: @winterqt, @lheckemann | ||
shepherd-leader: @lheckemann | ||
related-issues: (will contain links to implementation PRs) | ||
--- | ||
|
||
# Summary | ||
[summary]: #summary | ||
|
||
Develop and discuss RFCs in repositories instead of a pull request. | ||
|
||
# Motivation | ||
[motivation]: #motivation | ||
|
||
RFC discussions are currently held in single pull requests on the [rfcs](https://github.com/NixOS/rfcs/pulls) repository. | ||
Due to GitHub's limitations on pull requests, this quickly becomes unmanageable. | ||
The most prominent example of this is [the Flakes RFC](https://github.com/NixOS/rfcs/pull/49) with over 500 comments. | ||
|
||
# Detailed design | ||
[design]: #detailed-design | ||
|
||
The RFC process is changed to use repositories to develop and review RFCs instead. | ||
To create a new RFC, an issue on the [rfcs](https://github.com/NixOS/rfcs) repository is opened. | ||
It should contain the following contents describing the process: | ||
|
||
> This RFC is developed as the README.md of this repository: https://github.com/$owner/$repo | ||
> | ||
> To review the RFC | ||
> - Open https://github.com/$owner/$repo/blob/master/README.md?plain=1 | ||
> - Select the lines you wish to comment on | ||
> - Click the "..." menu on the left | ||
> - Select "Reference in new issue" to open a new issue commenting on those lines | ||
> | ||
> To suggest changes, open a PR against the repository. | ||
> | ||
> All issues and pull requests must be resolved before the FCP can be initiated and completed. | ||
> | ||
> This issue may only be used for RFC meta-discussions, such as shepherd nominations, FCP periods, meeting schedules, etc. | ||
|
||
The FCP must be initiated on a specific commit of the repository. | ||
When the FCP passes, the repositories contents are committed to the [rfcs](https://github.com/NixOS/rfcs) repository by the [RFC steering committee](https://github.com/NixOS/rfcs#rfc-steering-committee) with a commit that closes the original issue. | ||
|
||
## Workflow to get RFC updates | ||
To get updates for RFC's, instead of subscribing to the PR, one has to watch the repository. | ||
|
||
## Repository transfer | ||
|
||
Once the RFC is merged, the repository has to be transferred to the NixOS organization under https://github.com/NixOS/rfc-NUMBER. This is to ensure the discussions aren't lost in the future. The repository will then be archived. | ||
|
||
# Examples and Interactions | ||
[examples-and-interactions]: #examples-and-interactions | ||
|
||
|
||
# Drawbacks | ||
[drawbacks]: #drawbacks | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
This actually still works, because GitHub redirects I added the other point to the Drawbacks. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks, I added this to the drawbacks! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Shepherds already often end up making material contributions to the RFC text in the current process, as a fairly natural consequence of the process itself. IMHO: this is okay and doesn't hurt the spirit of the RFC process -- it still ensures that it's not just an author and a coauthor making decisions amongst themselves without anyone else in the community noticing. I think that if a team of 5 people has co-written an RFC it's perfectly fine for 3 of them to form the shepherd team (though any nominations outside the shepherd team should probably be given extra consideration by the steering committee). There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I noticed 2 unmentioned drawbacks (one of which I still don't remember), mentioning one of them here.
As for 1, I have a potential solution in mind; we can transfer the RFC repository to the NixOS org as a WIP RFC and protect the default branch so that all changes are visible in PRs as usual. Also, I really loved the idea of pre-RFCs, I think pre-RFCs could integrate well with this where the transfer would only occur once the pre-RFC has matured into a "normal" one. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The 2nd drawback I couldn't remember in my previous thread: If an RFC has a relatively large amount of traffic, multiple discussions can start on the same logical unit discussing more or less the same thing (or even the exact same thing). While this can be considered relatively trivial to manage, it's still an overhead that I thought needs to be mentioned (which apparently already was here: #138 (comment). This is a consequence of the previously mentioned absence of the line-comment view. Solution?A potential way of solving this can be to have a great label architecture (much like nixpkgs). We could have "broad" labels which could be: |
||
- There's no line-comment view anymore (the "Files changed" tab for PR's), where you can see the entire proposal and all (up-to-date) line comments. | ||
- Commenting on an RFC won't automatically subscribe you to updates anymore. | ||
- Less separation between RFC authors and reviewers with edit suggestions, making it harder to decide who may be a shepherd, since authors can't be shepherds. | ||
|
||
# Alternatives | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I suggest putting more focus on Pre-RFCs and on discussion in the forum. Rust does this and it works really well IMO. A lot of the discussion noise in GitHub threads currently comes to RFCs being posted as drafts containing typos (I've also occasionally witnessed this on Matrix RFCs too), or more generally from the first impact of an idea in the author's head with the reality of a community's opinions. A lot of that can be absorbed by iterating outside of the formal RFC process. I think then the discussion workload on GitHub would remain manageable. Even when keeping the approach of using GitHub repositories as suggested here, I'd still recommend putting more emphasis on pre-RFC discussions. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah that's a good suggestion. I think these two ideas go hand-in-hand: You can create a repository for development of the RFC beforehand, and if you want it to become more official, be looked at by more people, etc. open an issue in the rfcs repository linking to the repository where it's being developed. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe then a ful alternative could look like that:
I do think that splitting small enough not to need a separate repo is good when possible, and I think that submitting for the final review via the old process after drafting in a repo might be still reasonable otherwise. |
||
[alternatives]: #alternatives | ||
|
||
## Optional instead of required | ||
|
||
Instead of requiring this process, it could be opt-in for "bigger" RFCs. | ||
- (-) It's not clear how to decide whether an RFC should have a repository, there's no way to know how big discussions become in advance. | ||
|
||
## Fork branch instead of separate repository | ||
|
||
Instead of creating a new repository for each RFC, a new branch in a fork can be created instead. | ||
- (-) Means that a single GitHub user/organization can't have more than one RFC open at a time without mixing of issues/PRs occurring (since GitHub only supports having a single fork of a repository). | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this a big issue? Have there been instances in the past where the same person is authoring multiple RFCs simultaneously? And if so, would labels on both issues & PRs by RFC number not solve this problem satisfactorily? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unclear. Yes, e.g. 45/46. Maybe. To be fair, line comments get auto-obsoleted on updates but issues are not; so if a full rewrite test-wise (which 46 has seen) would need probably need labelling even within a single RFC. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm kind of already using this approach and I already have 4 RFC repos :)
I think it would be much cleaner to separate all RFCs with a repository. There's various benefits, like being able to infer that an RFC is likely ready for FCP once issues and PRs are at 0. Or not having to worry about having the correct label. Or the workflow not changing when somebody opens another RFC (before you wouldn't have needed a filter, but now you do), etc. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, that makes sense. Might not be worth exploring this alternative, in that case. |
||
- (+) For time-distinct RFCs it can be worked around by [detaching the old fork](https://support.github.com/request/fork) and creating a new one | ||
- (+) Simplifies the RFC process, since one can just create a PR to upstream it | ||
- (-) However this may again lead to the original problem of having too long PR discussions | ||
- (-) It's also confusing about whether a PR is needed | ||
|
||
# Unresolved questions | ||
[unresolved]: #unresolved-questions | ||
|
||
|
||
# Future work | ||
[future]: #future-work | ||
|
||
- More RFC automation is possible in the future: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the spirit of automation, I think there will be a strong need of a template repository for RFCs, especially if a label architecture would be enforced as I mentioned in this comment. |
||
- Creating repositories for discussing issues created in the rfcs repository | ||
- Assigning the shepherd team and requiring them to review PRs to the repository | ||
- Announce FCP when all issues/PRs are closed | ||
- Commit contents to rfcs repository once FCP passed without any new issues/PRs | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be done in a more centralized form.
This is merely an archive of discussions. Then a possible design is
https://github.com/NixOS/rfc-archives
https://github.com/NixOS/rfc-archives/tree/RFC-<number>
MERGED RFCS
section on README.MD (including a link to the branch created above)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The discussion happens in GitHub issues and pull requests though, it's not attached to Git itself. And while issues can be moved to separate repositories, moving discussions from separate RFCs to the same repository would create a very messy issue list.