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.

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

Travis ci improvements #357

Closed
wants to merge 6 commits into from
Closed

Conversation

martbab
Copy link
Contributor

@martbab martbab commented Dec 20, 2016

Test PR against my own fork to test Travis CI improvements.

Martin Babinsky added 6 commits December 20, 2016 10:11
this script is intended only for use in Travis CI and contains
configuration of the test run requested:

    * it can run linter step separately by specifying TASK_TO_RUN="lint"
      environment variable in .travis.yml. In this case it also runs
      pep8 checker on the commits in PR.
    * other steps are run in developer mode in order to skip pylint run
      and speed up the task
    * in all cases the CI result log is populated and can be displayed
      if the job fails
This prevents Travis log collector to add separate expansion marks to
the echo output and the actuall log output.
In order to speed our Travis CI gating even further, the lint step has
been split to a separate job that can be run in parallel with the test
runs. The test runs are in turn launched in developer mode to speed them
up.
@martbab martbab closed this Dec 20, 2016
@martbab martbab added the rejected Pull Request has been rejected label Dec 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rejected Pull Request has been rejected
Projects
None yet
1 participant