-
Notifications
You must be signed in to change notification settings - Fork 171
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
Create a FIP Editor Handbook for current and intending FIP Editors #1101
base: master
Are you sure you want to change the base?
Changes from all commits
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,86 @@ | ||
# FIP Editor Handbook | ||
|
||
## README | ||
This document serves as an extended guide on the roles, responsibilities, and expectations of FIP Editors. It builds upon the foundational description provided in **FIP-0001**, which introduced the FIP process and outlined the role of FIP Editors. While FIP-0001 offered an initial framework, this handbook provides more comprehensive guidance to ensure consistency, accountability, and clarity for FIP Editors and the community they serve. | ||
|
||
For detailed information on the FIP process and the foundational principles, refer to **FIP-0001** in the [Filecoin FIP repository](https://github.com/filecoin-project/FIPs). | ||
|
||
--- | ||
|
||
## Introduction | ||
Filecoin Improvement Proposal (FIP) Editors maintain the FIP repository and ensure the editorial completeness of proposals submitted on GitHub. They volunteer their time for the public good, supporting the FIP process by advancing proposals through various stages of their lifecycle. FIP Editors review new proposals, identify formatting and editorial issues, and keep the repository up to date. They leverage their technical expertise to benefit the community by reviewing improvement ideas and providing critical feedback. | ||
|
||
--- | ||
|
||
## Mission | ||
FIP Editors aim to facilitate the structured advancement of the Filecoin protocol by managing and refining FIP submissions. Their mission is to uphold the standards set by the community and ensure that each proposal receives a fair and thorough evaluation. | ||
|
||
--- | ||
|
||
## Responsibilities | ||
FIP Editors serve the community by: | ||
|
||
- **Editorial Review**: Ensuring proposals are well-formatted, technically complete, and adhere to Filecoin protocol standards. | ||
- **Facilitating Discussion**: Encouraging open dialogue and collaboration within the community to refine and enhance proposals. | ||
- **Upholding Quality**: Maintaining a standard of quality for proposals, focusing on the needs of their target audience. | ||
- **Repository Maintenance**: Managing version control and processing pull requests and FIPs in the GitHub repository promptly. | ||
- **Governance Support**: Identifying when proposals are ready to advance to the "Last Call" stage. | ||
- **Ensuring Review Standards**: Verifying that pull requests meet review standards before merging, as outlined below. | ||
|
||
--- | ||
|
||
## Limitations | ||
FIP Editors do not: | ||
|
||
- **Pass Judgments**: They do not serve as arbiters of the merit of proposals or determine their likelihood of acceptance. | ||
- **Assert Correctness**: While they may offer technical feedback, they do not confirm the technical soundness of proposals. | ||
- **Decide Network Upgrades**: They do not determine which FIPs should be included in network upgrades, even though some FIP Editors may be involved in implementation teams. | ||
|
||
--- | ||
|
||
## Pull Request Review Standards | ||
|
||
For pull requests (PRs) that impact **protocol design/specifications** or include **considerations** affecting the protocol: | ||
|
||
- FIP Editors must ensure **at least two appropriate technical reviews and approvals** have been obtained before considering the PR ready to merge. | ||
|
||
For **all other PRs**: | ||
luckyparadise marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- A minimum of **two technical reviews and approvals** is required before merging. | ||
|
||
### Additional Safeguards | ||
|
||
To ensure sufficient review time and prevent rushed decisions: | ||
|
||
- FIP Editors should allow a reasonable review period (e.g., 48–72 hours) after obtaining the required two approvals before merging the PR, unless the changes are time-sensitive and have explicit consensus from other editors. | ||
- Editors should use their judgment to balance the urgency of changes with the need for thorough review. | ||
|
||
--- | ||
|
||
## Membership | ||
|
||
Membership is open to all who wish to apply but is subject to consent by existing FIP Editors. This structure ensures scalable and controlled membership over time. As the process gains stability, existing FIP Editors may decide to make membership more open and less controlled. | ||
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 would be a change to how we do it now. Membership is currently by invitation and unanimous agreement (or abstention) of the existing editors. |
||
|
||
Candidates should meet the following criteria: | ||
|
||
- Commit **0–5 hours weekly** for reviewing FIPs. | ||
- Be proficient in English (both written and spoken). | ||
- Be capable of critiquing proposal concepts. | ||
- Be familiar with the **Filecoin tech stack**. | ||
- Have **participated in FIP discussions** (an added advantage). | ||
- Be interested in participating in governance. | ||
- Be skilled in using GitHub. | ||
Comment on lines
+64
to
+72
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 updating this to be more along the lines of if you would like to be invited to become a FIP editor than you should demonstrate these behaviours. |
||
|
||
Continued inability to meet these requirements may lead to removal from the position. Interested candidates can apply by completing [this form]. Current FIP Editors will review membership requests. | ||
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 there a link to the form someone submits? Are we actually going to make and maintain a form? Given how low volume this will be, is it better to create a mailing list (e.g., [email protected]) that contacts all the FIP editors and they handle from there? 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. Not even. I think we can just use Slack for this. This document doesn't need to specify a mechanism. If a candidate editor can't figure out how to contact the other editors, they're probably not a fit. |
||
|
||
--- | ||
|
||
## Decision Making | ||
|
||
### Informal Decisions | ||
For non-controversial decisions, particularly those affecting a single proposal, FIP Editors may select the course of action they deem fit, aligned with Filecoin's mission and values. | ||
|
||
### Formal Decisions | ||
|
||
- **Adding a FIP Editor**: Candidates must consent to the responsibilities and be unopposed by current editors. | ||
- **Removing a FIP Editor**: Voluntary resignation is preferred; however, involuntary removal follows documented failures and a chance for correction. A consensus among remaining editors, excluding the affected editor, is required to finalize removal, ensuring fairness and integrity in the process. |
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 suggest adjusting this to highlight that the technical expertise comes from the community.