Skip to content

azure pipelines config #715

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

Merged
merged 13 commits into from
May 13, 2019
Merged

azure pipelines config #715

merged 13 commits into from
May 13, 2019

Conversation

wholmgren
Copy link
Member

@wholmgren wholmgren commented May 8, 2019

Goal is to get both linux and windows building on azure pipelines and drop appveyor and eventually travis.

  • Closes appveyor builds failing due to solarposition with numba warnings #713
  • I am familiar with the contributing guidelines.
  • Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
  • Updates entries to docs/sphinx/source/api.rst for API changes.
  • Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
  • Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
  • New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
  • Pull request is nearly complete and ready for detailed review.

@wholmgren wholmgren added this to the 0.6.2 milestone May 9, 2019
@wholmgren
Copy link
Member Author

Ready to merge.

Azure Pipelines often completes 4 linux + 4 windows builds in under 6 minutes. This is less than 50% of TravisCI's time (albeit with 2 fewer linux configurations) and somewhere between 10-20% of the Appveyor's time (depending on how long until Appveyor actually gets started).

I'll turn off the Appveyor integration and delete .appveyor.yml once this is merged.

Dropping Travis is a little more complicated. Our Travis config has some special steps for the python 2.7 min build and I'd like to leave that alone until we drop 2.7. It also has a deploy key for pypi. Let's reevaluate in a few months.

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.

appveyor builds failing due to solarposition with numba warnings
1 participant