Skip to content

attempt-backport should ignore semver-major changes #105

Description

@mscdex

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

/cc @Fishrock123

Activity

  1. Fishrock123 commented on Dec 14, 2016

    @Fishrock123
    Contributor

    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?

  2. targos commented on Dec 14, 2016

    @targos
    Member

    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.

  3. phillipj commented on Dec 14, 2016

    @phillipj
    Member

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

  4. Fishrock123 commented on Dec 15, 2016

    @Fishrock123
    Contributor

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

  5. Fishrock123 commented on Dec 15, 2016

    @Fishrock123
    Contributor

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

  6. MylesBorins commented on Dec 15, 2016

    @MylesBorins
    Contributor

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

    edit: still better than nothing

  7. mscdex commented on Dec 28, 2016

    @mscdex
    ContributorAuthor

    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.

  8. Fishrock123 commented on Jan 4, 2017

    @Fishrock123
    Contributor

    @mscdex that is why #104 only removes it if the bot (last?) added it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions