You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.
A lot of people forget to fill the templates and send PRs for review without the proper information. Unfortunately, having the template means the check for description will always pass.
Exceptions:
We shouldn't try to apply it over all templates, but at least the one in Alice.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Proposal:
Modify the check for description at https://github.com/loadsmart/peril-settings/blob/master/rules/common/needsDescription.ts to consider our pre-made PR description templates as if it was empty.
Reasoning
A lot of people forget to fill the templates and send PRs for review without the proper information. Unfortunately, having the template means the check for description will always pass.
Exceptions:
We shouldn't try to apply it over all templates, but at least the one in Alice.
The text was updated successfully, but these errors were encountered: