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

MAINT: Drop redundant travis configs #5950

Merged
merged 1 commit into from
Jul 16, 2019

Conversation

bashtage
Copy link
Member

Drop configs that are run on appveyor

  • code/documentation is well formatted.
  • properly formatted commit message. See
    NumPy's guide.

@codecov
Copy link

codecov bot commented Jul 16, 2019

Codecov Report

Merging #5950 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5950      +/-   ##
==========================================
- Coverage   82.96%   82.96%   -0.01%     
==========================================
  Files         600      600              
  Lines       95064    95009      -55     
  Branches    10460    10444      -16     
==========================================
- Hits        78871    78821      -50     
+ Misses      13777    13771       -6     
- Partials     2416     2417       +1
Impacted Files Coverage Δ
statsmodels/tsa/stattools.py 86.79% <0%> (-1.25%) ⬇️
statsmodels/graphics/tests/test_mosaicplot.py 100% <0%> (ø) ⬆️
statsmodels/tools/grouputils.py 69.6% <0%> (+0.44%) ⬆️
statsmodels/regression/process_regression.py 77.61% <0%> (+1.16%) ⬆️

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 ae17fbd...ad6a650. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.001%) to 85.502% when pulling 8c9587b on bashtage:update-travis into 95d094f on statsmodels:master.

@coveralls
Copy link

coveralls commented Jul 16, 2019

Coverage Status

Coverage decreased (-0.01%) to 85.505% when pulling ad6a650 on bashtage:update-travis into ae17fbd on statsmodels:master.

@@ -74,14 +71,10 @@ matrix:
- os: osx
language: generic
env:
- PYTHON=3.6.6
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the travis pip version uses pyenv to get the python version and it fails because "3.6" is too ambiguous.

I noticed that the travis pip/openblas tended to be more delicate than the mkl version--especially for Mac. So I liked having separate ones. Not married to it, however.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can be dropped since python/pip is checked on azure for 3.5-3.7.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can be dropped since azure runs python/pip on OSX. The only issue here is that OpenBlas is tied to the NumPy version, so different NumPy will get different OpenBlas.

.travis.yml Outdated Show resolved Hide resolved
Drop configs that are run on appveyor
@bashtage bashtage merged commit 76b595e into statsmodels:master Jul 16, 2019
@bashtage bashtage deleted the update-travis branch July 16, 2019 08:15
@bashtage bashtage added this to the 0.11 milestone Dec 18, 2019
@bashtage bashtage changed the title MAINT: Drop redendant travis configs MAINT: Drop redundant travis configs Dec 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants