Skip to content

Revert "Skip AppVeyor builds on non-code changes" #5524

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
Jun 22, 2018

Conversation

pradyunsg
Copy link
Member

Reverts #5478
Reopens #5185

Reverted since AppVeyor is not reporting status for PRs when it's skipping builds.

@pradyunsg pradyunsg added type: bug A confirmed bug or unintended behavior C: automation Automated checks, CI etc type: maintenance Related to Development and Maintenance Processes labels Jun 21, 2018
@pradyunsg pradyunsg self-assigned this Jun 21, 2018
@pradyunsg pradyunsg added the skip news Does not need a NEWS file entry (eg: trivial changes) label Jun 21, 2018
@pradyunsg
Copy link
Member Author

#5491 is a docs-only PR where AppVeyor is misbehaving.

@pradyunsg pradyunsg merged commit 9ab837e into master Jun 22, 2018
@pradyunsg pradyunsg deleted the revert-5478-ci/short-circuiting-appveyor branch June 22, 2018 02:31
@lock
Copy link

lock bot commented Jun 2, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 2, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation C: automation Automated checks, CI etc skip news Does not need a NEWS file entry (eg: trivial changes) type: bug A confirmed bug or unintended behavior type: maintenance Related to Development and Maintenance Processes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant