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

Remove unused azure pipeline #65

Merged
merged 1 commit into from
Jun 14, 2022
Merged

Conversation

mscheltienne
Copy link
Member

@adam2392 This repo does not use azure, right?

@adam2392
Copy link
Member

We should! For windows testing.

@mscheltienne
Copy link
Member Author

Why the build and pytest workflows on the windows-latest runner from GitHub are not enough?

@adam2392
Copy link
Member

Oh actually that's a good question...

Didn't realize Windows was now supported. @larsoner do you know if Azure pipelines is still useful?

@larsoner
Copy link
Member

You can use Azure or GH actions for windows. GH Actions uses Azure under the hood IIRC anyway. So if you're happy with GH actions you can just go ahead and use it

@adam2392
Copy link
Member

Okay! @mscheltienne let's just get rid of it then.

Seems like the unit tests failed once the mne-testing-data was updated. Perhaps it's using a cached version of the data? Should we just re-run the CI?

@larsoner
Copy link
Member

No, downloading of the mne-testing-data releases is totally broken at the GH end. You'll need to test locally. Feel free to upvote this, maybe someone will notice

https://github.com/orgs/github-community/discussions/18543

@mscheltienne mscheltienne merged commit a411e45 into mne-tools:main Jun 14, 2022
@mscheltienne mscheltienne deleted the remove_azp branch June 14, 2022 15:24
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.

3 participants