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

Use correct Python versions in CI environments #658

Merged
merged 1 commit into from
Jan 5, 2021
Merged

Conversation

huddlej
Copy link
Contributor

@huddlej huddlej commented Jan 5, 2021

Currently, Travis CI runs one build for three major Python versions except each environment installs and uses the same Python 3.6 version from a fixed conda environment file. This change follows the recommendations from the conda documentation to build each conda environment with the Travis Python version variable and the conda create command instead of the conda env create command. The cost of duplicating our dependencies here is small compared to the benefit of testing Augur with the appropriate Python versions.

Fixes #626

Currently, Travis CI runs one build for three major Python versions
except each environment installs and uses the same Python 3.6 version
from a fixed conda environment file. This change follows the
recommendations from the conda documentation [1] to build each conda
environment with the Travis Python version variable and the `conda
create` command instead of the `conda env create` command. The cost of
duplicating our dependencies here is small compared to the benefit of
testing Augur with the appropriate Python versions.

Fixes #626

[1] https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/use-conda-with-travis-ci.html
@codecov
Copy link

codecov bot commented Jan 5, 2021

Codecov Report

Merging #658 (ff91b47) into master (7e31e18) will increase coverage by 0.32%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #658      +/-   ##
==========================================
+ Coverage   28.76%   29.09%   +0.32%     
==========================================
  Files          39       39              
  Lines        5409     5434      +25     
  Branches     1332     1332              
==========================================
+ Hits         1556     1581      +25     
  Misses       3795     3795              
  Partials       58       58              
Impacted Files Coverage Δ
augur/util_support/metadata_file.py 100.00% <0.00%> (ø)
augur/util_support/node_data_file.py 100.00% <0.00%> (ø)
augur/util_support/color_parser_line.py 100.00% <0.00%> (ø)
augur/frequency_estimators.py 33.84% <0.00%> (+0.12%) ⬆️
augur/util_support/color_parser.py 96.42% <0.00%> (+0.13%) ⬆️
augur/utils.py 36.64% <0.00%> (+0.19%) ⬆️
augur/util_support/node_data_reader.py 95.12% <0.00%> (+0.25%) ⬆️
augur/titer_model.py 18.90% <0.00%> (+0.29%) ⬆️
augur/util_support/date_disambiguator.py 94.11% <0.00%> (+0.36%) ⬆️
augur/util_support/shell_command_runner.py 91.48% <0.00%> (+0.79%) ⬆️

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 7e31e18...ff91b47. Read the comment docs.

@huddlej huddlej added this to the Patch release 10.2.1 milestone Jan 5, 2021
@huddlej huddlej merged commit aabef3c into master Jan 5, 2021
@huddlej huddlej deleted the fix-python-in-ci branch January 5, 2021 20:09
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.

Travis CI environments for Python 3.x all use Python 3.6
1 participant