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

docs: Use string rule severity in CLI examples

Merged
merged 1 commit into from Nov 5, 2021

Conversation

platinumazure
Copy link
Member

@platinumazure platinumazure commented Nov 4, 2021

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[x] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)

Replace numeric rule severity with string rule severity

Is there anything you'd like reviewers to focus on?

No

@eslint-github-bot eslint-github-bot bot added the triage label Nov 4, 2021
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 4, 2021

CLA Signed

The committers are authorized under a signed CLA.

nzakas
nzakas approved these changes Nov 5, 2021
Copy link
Member

@nzakas nzakas left a comment

LGTM

@nzakas nzakas added accepted documentation and removed triage labels Nov 5, 2021
Copy link
Member

@mdjermanovic mdjermanovic left a comment

LGTM, thanks!

@mdjermanovic mdjermanovic merged commit c415c04 into eslint:main Nov 5, 2021
14 checks passed
@platinumazure platinumazure deleted the cli-docs-rule-severity branch Nov 5, 2021
@eslint-github-bot eslint-github-bot bot locked and limited conversation to collaborators May 5, 2022
@eslint-github-bot eslint-github-bot bot added the archived due to age label May 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted archived due to age documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants