-
Notifications
You must be signed in to change notification settings - Fork 71
Description
Proposal
It is not allowed to merge changes referencing TODO, meaning that while working on a change, one can use TODO for things which still have to be fixed before merging the PR. It's nice that this means one can simply grep for TODO in the code base or at least in ./compiler.
There are currently the following uses of TODO:
rustc_codegen_cranelift: 5 usesrustc_codegen_gcc: 204 usesrustc_mir_buildas the text of a suggestion: https://github.com/rust-lang/rust/blob/e96bb7e44fbcc23c1e6009e8d0ee8ab208668fb4/compiler/rustc_mir_build/src/check_unsafety.rs#L121-L123
I would like us to change the uses in rustc_codegen_cranelift and rustc_codegen_gcc to FIXME and ideally add a CI check to these projects as well. cc @bjorn3 @antoyo I think.
There are a lot more uses in /library and /src. While I would like to do that change there as well, I am not proposing this as part of this MCP.
Mentors or Reviewers
If you have a reviewer or mentor in mind for this work, mention them here. You can put your own name here if you are planning to mentor the work.
Process
The main points of the Major Change Process are as follows:
- File an issue describing the proposal.
- A compiler team member who is knowledgeable in the area can second by writing
@rustbot secondor kickoff a team FCP with@rfcbot fcp $RESOLUTION.- Refer to Proposals, Approvals and Stabilization docs for when a second is sufficient, or when a full team FCP is required.
- Once an MCP is seconded, the Final Comment Period begins.
- Final Comment Period lasts for 10 days after all outstanding concerns are solved.
- Outstanding concerns will block the Final Comment Period from finishing. Once all concerns are resolved, the 10 day countdown is restarted.
- If no concerns are raised after 10 days since the resolution of the last outstanding concern, the MCP is considered approved.
You can read more about Major Change Proposals on forge.