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

ENH appveyor is ok list #2

Closed
wants to merge 7 commits into from
Closed

ENH appveyor is ok list #2

wants to merge 7 commits into from

Conversation

beckermr
Copy link
Member

@beckermr beckermr commented Apr 9, 2020

I've gone through the rest of the feedstocks that have appveyor.

The ones on this list currently use appveyor to build on master and have been worked on recently.

Everything else has been

  • moved to azure
  • uses appveyor on a version branch (e.g. python)
  • uses appveyor on a PR branch that should not be there
  • is a recipe that is 1.5 years to 4 years out of date

I have left these feedstocks off the list under the assumptions that

  1. should they ever get updated (either in a branch on master, we would rerender and move to azure)
  2. we can add any feedstock back later

Hopefully isl and libssh2 can be moved over to azure before this PR is merged.

cc @conda-forge/core for viz

here are my notes: https://hackmd.io/HSwAaR3ZTzSA70kSiPRwcg?edit

@beckermr
Copy link
Member Author

OK @conda-forge/core I've inspected the builds of the remaining packages. I removed anything that was either not maintained or had a build error that didn't appear to me to be related to azure versus appveyor. This left just isl in the list. I saw that @isuruf is working on moving the build of that to azure. So I think we should leave this issue open until we know about isl and then make a decision.

@beckermr beckermr changed the title ENH appveyor is ok list WIP appveyor is ok list Apr 14, 2020
@beckermr beckermr marked this pull request as draft April 14, 2020 13:10
@beckermr beckermr changed the title WIP appveyor is ok list ENH appveyor is ok list Apr 14, 2020
@beckermr
Copy link
Member Author

We moved isl due to hard work from @isuruf. I think we can just no longer support appveyor going forward. Closing!

@beckermr beckermr closed this Apr 14, 2020
@beckermr beckermr deleted the beckermr-patch-1 branch April 14, 2020 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant