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

Disable a couple of CI builds #2102

Conversation

ericpre
Copy link
Member

@ericpre ericpre commented Dec 1, 2018

At the moment, the appveyor build takes more 2h to complete, because we are testing 6 different builds and there are run sequentially. The situation is significantly with travis, which is running 5 builds at the same but I would also suggest to disable the python 3.5 and python 3.6 mac osx build to save CI time.

The idea is to continuously test python 3.5 and python 3.6 support with linux and to other disable build for a release when the build will be enabled to generate the wheels. This should be a good compromise between testing across different platform and python version and saving CI time.

On a side node, conda-forge is not packaging for python 3.5 anymore (only python 3.6 and python 3.7).

Progress of the PR

  • Disable python 3.5 build in appveyor,
  • enable one win32 build,
  • disable python 3.5 and python 3.6 on mac osx,
  • ready for review.

@francisco-dlp
Copy link
Member

Isn't it possible to run the disabled builds only on release instead of having to comments and uncomment the code manually?

@ericpre
Copy link
Member Author

ericpre commented Dec 5, 2018

I did look at it and could not a way to do but a 2nd iteration on this may be working! ;)

For reference:
https://www.appveyor.com/blog/2018/09/11/how-to-skip-build-matrix-jobs-conditionally/

@francisco-dlp
Copy link
Member

Could you create a tag in your repo (if we do it in this one it'll create a DOI in zenodo) to test if it's really working?

@ericpre ericpre mentioned this pull request Dec 5, 2018
6 tasks
@ericpre
Copy link
Member Author

ericpre commented Dec 5, 2018

@francisco-dlp
Copy link
Member

Isn't it possible to do the same in travis?

@ericpre
Copy link
Member Author

ericpre commented Dec 5, 2018

@ericpre ericpre force-pushed the disable_python35_test_appveyor branch from e5309bd to 8451306 Compare December 5, 2018 16:31
@francisco-dlp francisco-dlp merged commit 1158b95 into hyperspy:RELEASE_next_patch Dec 6, 2018
@ericpre ericpre deleted the disable_python35_test_appveyor branch December 6, 2018 09:55
@francisco-dlp francisco-dlp added this to the v1.4.2 milestone May 21, 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.

None yet

2 participants