github-bot attempt-backport should ignore semver-major changes

Title says it all, github-bot should not attempt to backport changes marked as semver-major.

/cc @Fishrock123

Dec 14 '16 08:12 mscdex

Should be fairly easy to do, but a lot will slip by because they will be labeled by us later so I'm not 100% sure what to do about that. Have any ideas?

Dec 14 '16 14:12 Fishrock123

a lot will slip by because they will be labeled by us later

Yes but at least once the PR is marked semver-major, the bot will stop trying on every push.

Dec 14 '16 14:12 targos

Maybe remove any previous backporting labels when GitHub emits a labelling event for semver-major being added?

Dec 14 '16 14:12 phillipj

@TheAlphaNerd Think it might work better if we only added these labels once a PR is closed/merged?

Dec 15 '16 17:12 Fishrock123

That does mean that people won't know if they'll need to adjust it until after though... :/ So IMO would be worse.

Dec 15 '16 17:12 Fishrock123

I'm ok with this. As long as it is accurate.

edit: still better than nothing

Dec 15 '16 17:12 MylesBorins

So does everyone agree that the bot should (at least) not add any backport-related labels if it sees a change in commits and a semver-major is set on the PR?

I'm not sure proactively removing existing backport-related labels would be a good thing, especially if someone intentionally added the label.

Dec 28 '16 00:12 mscdex

@mscdex that is why https://github.com/nodejs/github-bot/pull/104 only removes it if the bot (last?) added it.

Jan 04 '17 20:01 Fishrock123