FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Kill the AppVeyor file whitelist by zware · Pull Request #5121 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

Kill the AppVeyor file whitelist - #5121

Merged
zware merged 1 commit into
masterfrom
kill-appveyor-whitelist
Jan 7, 2018
ConversationCommits (1)ChecksFiles changed
Merged

Kill the AppVeyor file whitelist#5121
zware merged 1 commit into
masterfrom
kill-appveyor-whitelist

Conversation

zware commented Jan 7, 2018

Copy link
Copy Markdown
Member

It's more trouble than it's worth, since it only checks the HEAD commit of a PR rather than the full diff against the base branch for changed files.

Kill the AppVeyor file whitelist
05eef30 
It's more trouble than it's worth, since it only checks the HEAD commit of a PR rather than the full diff against the base branch for changed files.
zware merged commit 7f7de37 into master Jan 7, 2018

miss-islington commented Jan 7, 2018

Copy link
Copy Markdown
Contributor

Thanks @zware for the PR 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6.
🐍🍒⛏🤖

zware deleted the kill-appveyor-whitelist branch January 7, 2018 02:41

miss-islington commented Jan 7, 2018

Copy link
Copy Markdown
Contributor

Sorry, @zware, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 7f7de371f947dc38e67505601927e9bc58fa268a 3.6

miss-islington commented Jan 7, 2018

Copy link
Copy Markdown
Contributor

Sorry, @zware, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 7f7de371f947dc38e67505601927e9bc58fa268a 2.7

zware added a commit to zware/cpython that referenced this pull request Jan 7, 2018
[3.6] Kill the AppVeyor file whitelist (pythonGH-5121)
249a465 
It's more trouble than it's worth, since AppVeyor only checks the HEAD commit of a PR rather than the full diff against the base branch to decide which files changed..
(cherry picked from commit 7f7de37)

bedevere-bot commented Jan 7, 2018

Copy link
Copy Markdown

GH-5122 is a backport of this pull request to the 3.6 branch.

zware added a commit to zware/cpython that referenced this pull request Jan 7, 2018
[2.7] Kill the AppVeyor file whitelist (pythonGH-5121)
d26b888 
It's more trouble than it's worth, since AppVeyor only checks the HEAD commit of a PR rather than the full diff against the base branch to decide which files changed..
(cherry picked from commit 7f7de37)

bedevere-bot commented Jan 7, 2018

Copy link
Copy Markdown

GH-5123 is a backport of this pull request to the 2.7 branch.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL