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.

Sign up for GitHub

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

Jump to bottom

remove global-install eslint in cli docs #15498

Closed
1 task
aladdin-add opened this issue Jan 8, 2022 · 1 comment · Fixed by #15513
Closed
1 task

remove global-install eslint in cli docs #15498

aladdin-add opened this issue Jan 8, 2022 · 1 comment · Fixed by #15513
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion documentation Relates to ESLint's documentation enhancement This change enhances an existing feature of ESLint
Projects
Triage

Comments

@aladdin-add
Copy link
Member

aladdin-add commented Jan 8, 2022 •
edited

ESLint version

main

What problem do you want to solve?

https://github.com/eslint/eslint/blob/main/docs/user-guide/command-line-interface.md#command-line-interface

In general we should not recommend global-usage.

What do you think is the correct solution?

just remove the first para, and add a link to getting-started.md#installation-and-usage. thoughts?

Participation

  • I am willing to submit a pull request for this change.

Additional comments

No response
@aladdin-add aladdin-add added enhancement This change enhances an existing feature of ESLint triage An ESLint team member will look at this issue soon core Relates to ESLint's core APIs and features labels Jan 8, 2022
@eslint-github-bot eslint-github-bot bot added this to Needs Triage in Triage Jan 8, 2022
@aladdin-add aladdin-add added documentation Relates to ESLint's documentation and removed triage An ESLint team member will look at this issue soon core Relates to ESLint's core APIs and features labels Jan 8, 2022
@aladdin-add aladdin-add moved this from Needs Triage to Feedback Needed in Triage Jan 8, 2022
@mdjermanovic mdjermanovic added the accepted There is consensus among the team that this change meets the criteria for inclusion label Jan 12, 2022
@mdjermanovic mdjermanovic moved this from Feedback Needed to Pull Request Opened in Triage Jan 12, 2022
@mdjermanovic
Copy link
Member

PR #15513

@mdjermanovic mdjermanovic linked a pull request Jan 12, 2022 that will close this issue
docs: Update CLI docs to prefer local install #15513
Merged
1 task
@nzakas nzakas mentioned this issue Jan 13, 2022
Merged
1 task
Triage automation moved this from Pull Request Opened to Complete Jan 15, 2022
@eslint-github-bot eslint-github-bot bot locked and limited conversation to collaborators Jul 15, 2022
@eslint-github-bot eslint-github-bot bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Jul 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion documentation Relates to ESLint's documentation enhancement This change enhances an existing feature of ESLint
Projects
Triage
Archived in project
Triage
Complete
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: Update CLI docs to prefer local install eslint/eslint
2 participants
@aladdin-add @mdjermanovic