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: show console call with no-restricted-syntax (fixes #7806) #9376
Docs: show console call with no-restricted-syntax (fixes #7806) #9376
Conversation
LGTM |
LGTM |
docs/rules/no-console.md
Outdated
}; | ||
``` | ||
|
||
With the `no-console` rule, this will receive a warning/error. For the above example, you can disable the rule: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the
no-console
rule
=> With the no-console
rule enabled? not sure, just checking.. (°°)~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clarified with an example rule configuration so that an error would be reported
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for contributing!
[X] Documentation update
What changes did you make? (Give an overview)
Add documentation on how to show warnings/error for console calls and not method overwrites with the use of no-restricted-syntax
Is there anything you'd like reviewers to focus on?
That I got all the details from the opened issue