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

Clarify/Outline the RFC Withdrawal Process & Amendment #135

Merged
merged 1 commit into from
May 20, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 15 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
# npm RFC Process
# RFC Process

This document describes the RFC process for the [npm
CLI](https://github.com/npm/cli), and provides a way for the [Community & Open Source Team]() and the
CLI](https://github.com/npm/cli), and provides a way for the [Community & Open Source Team](https://github.com/orgs/npm/teams/cli-team) and the
wider community to have discussions about the features and direction of the
package manager! It is based on [the WeAllJS RFC process](https://wealljs.org/rfc-process) and the [Rust RFC process](https://github.com/rust-lang/rfcs), and, by extension, the [Yarn RFC process](https://github.com/yarnpkg/rfcs)

## What's an RFC?

The name itself is a reference to the IETF's Request For Comments process, and
The name itself is a reference to the **IETF's Request For Comments** process, and
basically involves a document or series of documents which are drafted,
reviewed, and eventually ratified (approved) by the npm team through discussion
among those interested, both within and outside of the npm team.
Expand Down Expand Up @@ -78,11 +78,22 @@ integration into the CLI may also be deferred to a later date, or a later
semver-major CLI release, at the npm collaborators' discretion. All the RFC does
is communicate the team's consensus to accept a change.

### Implementation

When the changes described in an RFC have been implemented and merged into the
relevant repository (and thus, due to be released), the corresponding RFC will
be moved from `accepted/` to `implemented/`. If you'd like to implement an
accepted RFC, please make a PR in the appropriate repo and mention the RFC in
the PR. Feel free to do this even for work-in-progress code!
the PR. Feel free to do this even for work-in-progress code.

### Withdrawal

From time to time npm collaborators will review RFCs awaiting implementation to
ensure their accuracy and relevance. In cases where a previously ratified RFC is
deemed to no longer be a viable candidate for implementation, an [**amendment
section**](withdrawn/0000-template.md) will be added **to the top** of the
document outlining the reason for repeal and subsequently moved to the
Copy link
Contributor

Choose a reason for hiding this comment

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

My only nit is that the amendment section should be at the top, not the bottom, so that it's harder to miss.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍🏻 Updated

`withdrawn/` section of this repository.

## How do I change an RFC after ratification?

Expand Down
9 changes: 9 additions & 0 deletions withdrawn/0000-template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# Withdrawal Amendment

{{This section is required for withdrawing a previously ratified RFC. Provide as much detail as possible as to why the originally drafted proposal no longer should be considered for implementation}}

## Relevant Resources

{{Optional section to link relevant resources and discussions relevant to this RFCs withdrawal}}

{{Original RFC}}