Charcoal-SE / SmokeDetector

Headless chatbot that detects spam and posts links to it in chatrooms for quick deletion.
https://metasmoke.erwaysoftware.com
Apache License 2.0
474 stars 182 forks source link

Make append-only files easier to merge #776

Closed tripleee closed 6 years ago

tripleee commented 7 years ago

If a blacklist PR sits around for a while, it is likely to conflict with subsequent commits. This is rather silly, because the files are basically append-only, and the conflicts are trivial to resolve.

http://stackoverflow.com/questions/34795604/does-git-support-automatic-merge-conflict-resolution-for-append-only-files outlines a way to tell Git how to resolve these conflicts automatically. I'm proposing to take it into use.

Undo1 commented 7 years ago

Makes sense to me. Curious whether GitHub will respect that setting, though.

quartata commented 6 years ago

This can be closed now (didn't automatically close since the PR was on the Metasmoke side)