Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Disable CI for documentation only changes #3294

Closed
sarum9in opened this issue Mar 22, 2018 · 1 comment
Closed

Disable CI for documentation only changes #3294

sarum9in opened this issue Mar 22, 2018 · 1 comment
Assignees
@sarum9in
Labels
CI Continuous integration: Travis/Appveyor/Actions

Comments

@sarum9in
Copy link
Contributor

Probably fail-fast approach can work. @jpakkane noted that in the past an attempt was made to implement this but coverage was broken because of this, so let's be careful this time.
@sarum9in sarum9in added the CI Continuous integration: Travis/Appveyor/Actions label Mar 22, 2018
@sarum9in sarum9in self-assigned this Mar 22, 2018
@sarum9in sarum9in mentioned this issue Mar 22, 2018
Merged
@QuLogic
Copy link
Member

QuLogic commented Mar 23, 2018

The original problem was that it was skipped on AppVeyor only. If you can skip both, the coverage should be fine.

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

@sarum9in sarum9in

Labels
CI Continuous integration: Travis/Appveyor/Actions
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@QuLogic @sarum9in