Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

[Drone 0.5] Use branch exclusion rule to avoid running tests on CI for documentation changes #1141

Closed
shuklanirdesh82 opened this issue Apr 3, 2017 · 2 comments

Comments

@shuklanirdesh82
Copy link
Contributor

shuklanirdesh82 commented Apr 3, 2017

Avoid running tests on CI for documentation change using branch exclusion rule [depends on drone 0.5 release]

Currently CI is running for documentation change (e.g. README.md, CONTRIBUTING.md etc) even though there is no code change.

CI should not be invoked when there is a doc change.

/cc @tusharnt

@shuklanirdesh82
Copy link
Contributor Author

shuklanirdesh82 commented Apr 28, 2017

Avoiding CI run can be achieved in two ways.

shuklanirdesh82 added a commit to shuklanirdesh82/vsphere-storage-for-docker that referenced this issue May 3, 2017
@shuklanirdesh82 shuklanirdesh82 changed the title Avoid running tests on CI for documentation change Avoid running tests on CI for documentation change using branch exclusion rule [depends on drone 0.5 release] May 12, 2017
@shuklanirdesh82 shuklanirdesh82 modified the milestones: Future, v1 GA May 12, 2017
@shuklanirdesh82 shuklanirdesh82 changed the title Avoid running tests on CI for documentation change using branch exclusion rule [depends on drone 0.5 release] [Drone 0.5] Use branch exclusion rule to avoid running tests on CI for documentation changes Jul 8, 2017
@shuklanirdesh82
Copy link
Contributor Author

let's stick with simpler approach that means "Adding [SKIP CI] or [CI SKIP] at PR title i.e. first commit message while generating PR (##1213)"

required work is done, closing this issue as wontfix; please feel free to reopen with some details about the reason for not using simpler approach.

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