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
Add Flow reminder to PR template #8805
Conversation
We also need to add this step:
Could you add this too, and do it both in PR template and here? |
5. Make sure your code lints (`npm run lint`). | ||
6. Run the [Flow](https://flowtype.org/) typechecks (`npm run flow`). | ||
7. If you haven't already, complete the CLA. | ||
3. If you added or removed any tests, run ./scripts/fiber/record-tests before submitting the pull request, and commit the resulting changes. |
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.
Please enclose the command in backticks.
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.
Also, this step should probably come the last before CLA. It should be done after everything else is ready.
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.
That makes more sense, I've made the changes!
Thanks! |
* Added flow to PR template * Added record-tests step to PR template and contribution docs * Updated order of PR checks (cherry picked from commit bfd5b18)
Since CircleCI fails on Flow errors, I think it would be helpful to add a reminder to
npm run flow
before submitting a PR.Formatting was inspired by the Draft.js PR template.