-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fix pull request template's markdown, reduce the required content #385
Comments
We should probably introduce the markdown validation check, too. |
I wonder whether the "Best practices" section is something that should be rethinked (or removed even), as it currently only provides a summary for accessibility. There are basically million best practices one could list here, and like the accessibility things, they should already be required on the definition of done of everything. |
I totally agree. PR template should be really minimal, precise and not an obstacle as it's now (at least for me). |
- Removed 'Best practices' section for being redundant - Add square brackets for all the texts that should be replaced
The current pull request template has too many elements and content. Let's try to reduce it to a reasonable level.
Also, there are several syntax issues with the pull request template markdown:
The text was updated successfully, but these errors were encountered: