JJ / github-pr-contains-action

Action that checks whether the body or diff in a PR contains a certain word.
MIT License
29 stars 33 forks source link

Improve documentation #137

Open JJ opened 3 months ago

JJ commented 3 months ago

This is always necessary, but since version 14.1 if you include something related to diffs, it will launch those tests, so I would like to avoid patterns such as this one where unused variables are set to a null string. In JS, that's pretty much the same as null, but anyway, it should be clear that you don't need to set variables to a null string to null them, since there are no defaults.