Skip to content
This repository has been archived by the owner on Mar 5, 2023. It is now read-only.

Enhancement: Do not trigger Travis CI/AppVeyor tests for document only changes in PR #813

Open
tshradheya opened this issue Jan 25, 2018 · 1 comment
Labels

Comments

@tshradheya
Copy link
Member

tshradheya commented Jan 25, 2018

Considering that a lot of students use the AB4 for the CS2103 project and the Travis tests take up to 5-6 minutes, it is not a good idea to make the tests run when the student has made document changes.

This will also prevent high traffic for Travis builds which was experienced previous semester.

Credit for idea: TEAMMATES/teammates#6432

@yamgent
Copy link
Member

yamgent commented Jan 25, 2018

Nice suggestion, thanks for raising it up.

I checked the build script and I think we can exclude these gradle tasks for documentation-only changes:

  • checkstyleMain
  • checkstyleTest
  • headless allTests
  • coverage
  • coveralls

I chose to exclude only these few specific tasks, because the rest serves non-tests purposes (for example, ./gradlew asciidoctor actually generates the documentation files for web deployment, run-checks.sh runs checks on files that are not codes as well, and etc...)

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

No branches or pull requests

3 participants