-
Notifications
You must be signed in to change notification settings - Fork 226
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
Update CONTRIBUTING.md #4733
Update CONTRIBUTING.md #4733
Conversation
| `pnpm format:check` | Validate files are formatted | | ||
| `pnpm gen-extern-signature` | Regenerate TypeScript signature for decorators(except compiler) | | ||
| `pnpm change add` | Add a change description | | ||
| `pnpm lint` | Run linters | | ||
| `pnpm lint:fix` | Fix autofixable issues | | ||
| `pnpm regen-samples` | Regen the samples(when the samples test fail) | | ||
| `pnpm regen-docs` | Regen the reference docs | | ||
| `pnpm run setup:min` `pnpm run format` | Run the formatter to reformat all files | |
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.
The setup command needs to be run before pnpm run format can be run :)
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.
This is specific to the isolated packages though. All those workspace command assume you have built the workspace.
mayhe we can categorize which one only need the min setup and which one the full build
No changes needing a change description found. |
You can try these changes here
|
@nisha-bhatia Closing since there is an open conversation and his is over a month old. |
No description provided.