Skip to content

Test ETS packages from source using cron jobs #162

Merged
rahulporuri merged 4 commits into
masterfrom
ci/travis-cron
Jun 24, 2019
Merged

Test ETS packages from source using cron jobs #162
rahulporuri merged 4 commits into
masterfrom
ci/travis-cron

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

The changes introduced in this PR are based on similar changes in the PR enthought/traits#479

Essentially, we do two things here

  • update the install command to install certain packages from source, via the use of a --source command line flag
  • update travis to use the --source flag when the event type is a cron job.

Poruri Sai Rahul added 3 commits June 22, 2019 15:49
and update travis to only install packages from source if the event type
is a cron job

	modified:   .travis.yml
	modified:   etstool.py
This is to test the behavior of the install command when the event type
is a cron job

	modified:   .travis.yml
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jun 22, 2019

Codecov Report

Merging #162 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #162   +/-   ##
=======================================
  Coverage   39.59%   39.59%           
=======================================
  Files         155      155           
  Lines        4592     4592           
  Branches      519      519           
=======================================
  Hits         1818     1818           
  Misses       2734     2734           
  Partials       40       40

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 30512d9...102398e. Read the comment docs.

@rahulporuri
Copy link
Copy Markdown
Contributor Author

As expected, pyface, traits and traitsui were installed from source in the travis job. See relevant exceprt from the job log below

[EXECUTING] edm plumbing remove-package --environment envisage-test-3.5-null --force pyface
[EXECUTING] edm plumbing remove-package --environment envisage-test-3.5-null --force traits
[EXECUTING] edm plumbing remove-package --environment envisage-test-3.5-null --force traitsui
[EXECUTING] edm run -e envisage-test-3.5-null -- python -m pip install git+http://github.com/enthought/pyface.git#egg=pyface --no-deps
Collecting pyface from git+http://github.com/enthought/pyface.git#egg=pyface
  Cloning http://github.com/enthought/pyface.git to /tmp/pip-install-ta053ejq/pyface
Installing collected packages: pyface
  Running setup.py install for pyface ... //done
Successfully installed pyface-6.2.0.dev19
[EXECUTING] edm run -e envisage-test-3.5-null -- python -m pip install git+http://github.com/enthought/traits.git#egg=traits --no-deps
Collecting traits from git+http://github.com/enthought/traits.git#egg=traits
  Cloning http://github.com/enthought/traits.git to /tmp/pip-install-bq5pxsku/traits
Installing collected packages: traits
  Running setup.py install for traits ... -\done
Successfully installed traits-5.2.0.dev59

The --source cmdline flag was set for regular CI jobs earlier to test
the behavior when the event type is a cron job. Now that the expected
behavior has been verified, we remove the flag

	modified:   .travis.yml
@rahulporuri rahulporuri requested a review from mdickinson June 22, 2019 13:23
@mdickinson
Copy link
Copy Markdown
Member

Nice. LGTM.

@mdickinson
Copy link
Copy Markdown
Member

@rahulporuri Please merge, but please remember to use squash and merge.

@rahulporuri rahulporuri merged commit 19f2431 into master Jun 24, 2019
@rahulporuri rahulporuri deleted the ci/travis-cron branch June 24, 2019 07:05
@rahulporuri
Copy link
Copy Markdown
Contributor Author

FTR i'm not sure when it was added but there already is a cron job running weekly on master.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants