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
Currently, if your pull request has a bracket in it (e.g. "ci: upgrade black (fixes #1621)") then gitlint fails.
I rather like having brackets in pull requests to denote such fixes. can we fix how we pass the commit message to gitlint so this will work going forwards?
The text was updated successfully, but these errors were encountered:
terriko
changed the title
gitlint can't handle brackets
Sanitize pull request title before passing to gitlint
Mar 29, 2022
It's a little embarrassing that I didn't notice this earlier since I'm literally trained for this kind of error, but since I have to approve pull requests until a new contributor has code merged, at least only existing contributors could have taken advantage of this (and it would have been reasonably obvious in the PR logs). I've merged a fix already.
Currently, if your pull request has a bracket in it (e.g. "ci: upgrade black (fixes #1621)") then gitlint fails.
I rather like having brackets in pull requests to denote such fixes. can we fix how we pass the commit message to gitlint so this will work going forwards?
The text was updated successfully, but these errors were encountered: