Skip to content

Ask Appveyor to ignore certain branches. #9286

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

Carreau
Copy link
Contributor

@Carreau Carreau commented Oct 5, 2017

This should prevent the automatic backports to be tested both when the
branch is pushed by the backport bot, and when the PR is opened.

Hopefully the PR will be still be tested (that's unclear from the
AppVeyor docs).

This was discussed in
#9283 (comment)

And likely need to be also applied to other CI, but Appveyor is the most
annoying.

From this page:
https://www.appveyor.com/docs/branches/

I'm guessing this will have to be backported to all relevant branches.

cc @QuLogic @anntzer and @tacaswell

This should prevent the automatic backports to be tested both when the
branch is pushed by the backport bot, and when the PR is opened.

Hopefully the PR will be still be tested (that's unclear from the
AppVeyor docs).

This was discussed in
matplotlib#9283 (comment)

And likely need to be also applied to other CI, but Appveyor is the most
annoying.
@QuLogic
Copy link
Member

QuLogic commented Oct 5, 2017

Should've named this branch auto-backport-* to test. 😉

@Carreau
Copy link
Contributor Author

Carreau commented Oct 5, 2017

Should've named this branch auto-backport-* to test.

Lol.

Still I can't push to MPL repo, so I guess it wouldn't have worked anyway :-)

@tacaswell tacaswell added this to the 2.1.1 (next bug fix release) milestone Oct 5, 2017
@tacaswell tacaswell merged commit 589b953 into matplotlib:master Oct 5, 2017
tacaswell added a commit that referenced this pull request Oct 5, 2017
@Carreau Carreau deleted the skip-autobackport branch October 5, 2017 19:29
@tacaswell
Copy link
Member

@meeseeksdev backport to v2.1.0-doc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants