EmbarkStudios / octobors

Rust program for automerging PRs based on a few rules
Apache License 2.0
40 stars 2 forks source link

Remove html comments from PR's body before merge. #28

Closed bnjbvr closed 2 years ago

bnjbvr commented 2 years ago

Checklist

Description of Changes

Most HTML code comments in PR texts are remains of PR templates that get embedded into the PR's commit message; that may pollute commit logs in the future, if users aren't careful enough and don't remove those manually. This PR proposes a very bespoke hacky method to remove them from the PR's text just before an attempt to merge.

Related Issues

n/a

davidpdrsn commented 2 years ago

I'll merge and deploy this :)