-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Initial pass removing old linting documentation #6193
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
Conversation
Co-authored-by: Courtney Webster <[email protected]>
Co-authored-by: Luciana Abud <[email protected]>
Co-authored-by: Courtney Webster <[email protected]>
@karthiknadig For the question about a new topic or just update the existing, I would say update the existing topic with the currently available and recommended settings. |
Co-authored-by: Luciana Abud <[email protected]>
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! thank you so much for working on this 😊
@karthiknadig @luabud I did a quick edit/formatting pass and commit so the PR will need another approval before it can be merged. Looks good! |
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!
This is linting documentation for linting using the new extensions. The old page that this is replacing is still valid, but we are deprecating it starting milestone of June 2023.
@gregvanl What do you recommend, do we create a different page for this or replace the existing one with this? The old settings will be valid till we remove it from the extension which is planned for last quarter of this year.