-
-
Notifications
You must be signed in to change notification settings - Fork 71
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
8ff6fec
commit a8643eb
Showing
1 changed file
with
140 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,140 @@ | ||
- Repo: [eslint/eslint](https://github.com/eslint/eslint) | ||
- Start Date: 2020-06-29 | ||
- RFC PR: | ||
- Authors: Aniketh Saha ([anikethsaha](https://github.com/anikethsaha)) aladdin-add([email protected]) | ||
|
||
# Move --init flag into a separate utility | ||
|
||
## Summary | ||
|
||
- Move the `init` command from main repo ([eslint](https://github.com/eslint/eslint)) to a new repo named `@eslint/create`. | ||
- Remove the auto-config. | ||
|
||
## Motivation | ||
|
||
Currently the whole `init` command is being shipped with the cli in the main repo. Though its not that much of size (roughly ~0.6MB for the [eslint/lib/init](https://github.com/eslint/eslint/tree/master/lib/init)), this command is not a type of command that we need everyday or everytime running eslint. It is mainly used when creating a new project or adding eslint to a project for the first time. So if we move this to a separate package that is meant to use in the command line, | ||
We will make use of tools like `npx` to simply run `npx @eslint/create` or using `npm` to run `npm init @eslint` or using `yarn` to run `yarn create @eslint` single time for a project instead of having it in the core project. | ||
|
||
## Detailed Design | ||
|
||
<!-- | ||
This is the bulk of the RFC. | ||
Explain the design with enough detail that someone familiar with ESLint | ||
can implement it by reading this document. Please get into specifics | ||
of your approach, corner cases, and examples of how the change will be | ||
used. Be sure to define any new terms in this section. | ||
--> | ||
|
||
The implementation is mainly migrating the code from main repo to `@eslint/create`. As far as the modules are concerned, all shared modules are being shipped currently with the eslint so we can use those modules through eslint itself. | ||
|
||
The approach would be following these steps | ||
|
||
### Move `eslint --init` related files to a separate repo | ||
|
||
- Move the `eslint/lib/init/*` to the new repo's `lib/*` directory | ||
- Add the following `dependencies` in `package.json`(the last 3 dependencies will be removed in next step) | ||
|
||
- `enquirer` | ||
- `progress` | ||
- `semver` | ||
- `lodash` | ||
- `json-stable-stringify-without-jsonify` | ||
- `cross-spawn` | ||
- `debug` | ||
- `@eslint/eslintrc` (to be removed) | ||
- `eslint` (to be removed) | ||
- `espree` (to be removed) | ||
|
||
- for all those modules coming inside `../**/*`, it would be replaced using `eslint/**/*` | ||
- For `tests`, move the `tests/lib/init` to new repo's `tests/init` | ||
- Add the following `devDependencies` in `package.json` | ||
|
||
- `chai` | ||
- `sinon` | ||
- `js-yaml` | ||
- `proxyquire` | ||
- `shelljs` | ||
|
||
It was almost done: [aladdin-add/eslint-create](https://github.com/aladdin-add/eslint-create). | ||
I will transfer the repo to the eslint group later. | ||
|
||
### Remove eslint auto-config | ||
|
||
- Remove auto-config(`eslint/lib/init/autoconfig.js`), and its tests(`eslint/tests/lib/init/autoconfig.js`) | ||
|
||
### Update eslint cli usage | ||
|
||
- Whenever `--init` command is being used, show a warning(e.g. "You can also run this command directly using 'npm init @eslint'") and run `npx @eslint/create` using `child_process` of the native node modules. | ||
|
||
|
||
## Documentation | ||
|
||
<!-- | ||
How will this RFC be documented? Does it need a formal announcement | ||
on the ESLint blog to explain the motivation? | ||
--> | ||
|
||
- Documentation for `@eslint/create` will be created with proper usage and each prompt's details. In the main documentation, a link to `@eslint/create` will be given [here](https://github.com/eslint/eslint/blob/master/docs/user-guide/command-line-interface.md#--init) and basic usage and the package details will be documented. | ||
Also in the cli command options, [here](https://github.com/eslint/eslint/blob/master/docs/user-guide/command-line-interface.md#options), for `--init` we need to change the description. | ||
- It seems a breaking change to remove auto-config, a formal announcement needed? | ||
This is a semver-minor change. | ||
|
||
## Drawbacks | ||
|
||
<!-- | ||
Why should we *not* do this? Consider why adding this into ESLint | ||
might not benefit the project or the community. Attempt to think | ||
about any opposing viewpoints that reviewers might bring up. | ||
Any change has potential downsides, including increased maintenance | ||
burden, incompatibility with other tools, breaking existing user | ||
experience, etc. Try to identify as many potential problems with | ||
implementing this RFC as possible. | ||
--> | ||
|
||
- This might increase the maintenance burden as this will add one more repo in the organization. | ||
- We may need to face challenges like re-directing of issues from `@eslint/create` repo to `eslint` and vice-versa. | ||
|
||
## Backwards Compatibility Analysis | ||
|
||
<!-- | ||
How does this change affect existing ESLint users? Will any behavior | ||
change for them? If so, how are you going to minimize the disruption | ||
to existing users? | ||
--> | ||
|
||
- Users can use `eslint --init`, which is as the same as current. | ||
- Users can no longer use auto-config. | ||
|
||
## Alternatives | ||
|
||
<!-- | ||
What other designs did you consider? Why did you decide against those? | ||
This section should also include prior art, such as whether similar | ||
projects have already implemented a similar feature. | ||
--> | ||
|
||
The current implementation in `eslint` repo is the alternative. No changes needed. | ||
|
||
## Open Questions | ||
|
||
<!-- | ||
This section is optional, but is suggested for a first draft. | ||
What parts of this proposal are you unclear about? What do you | ||
need to know before you can finalize this RFC? | ||
List the questions that you'd like reviewers to focus on. When | ||
you've received the answers and updated the design to reflect them, | ||
you can remove this section. | ||
--> | ||
|
||
- Do we want to implement this as monorepo under `eslint` repo or a separate repo `@eslint/create` but github will make it as `eslint-create` or similar ? | ||
|
||
We decided not to go in the monorepo direction for this. | ||
|
||
- What should be the name of the package/repo? | ||
|
||
We got a consensus on @eslint/create and eslint-create. |