-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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-beta] README change regarding lint:fix #23858
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @hellendag and the rest of your teammates on Graphite |
Deploy preview for dagster-docs-beta ready! ✅ Preview Built with commit 4d49b80. |
ff96069
to
875c116
Compare
875c116
to
4d49b80
Compare
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.
ty!
@@ -63,15 +63,12 @@ This command starts a local development server and opens up a browser window. Mo | |||
To check the documentation for different issues, use the following: | |||
|
|||
```bash | |||
## Lints Markdown content using prettier | |||
## Lints all content, applies lint autofixes and prettier changes | |||
yarn lint |
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.
i think this breaks our CI right? since we have a GH action that ran prettier --check and eslint without fix
Summary & Motivation
Modified yarn commands. Now:
ts
runs TypeScriptlint
applies prettier changes and autofixesvale
checks valuelint-and-vale
is a combination of the two aboveHow I Tested These Changes
Run the commands, verify that they behave as expected.
Changelog [New | Bug | Docs]
NOCHANGELOG