Skip to content

CI: Move from AppVeyor to Azure Pipelines #783

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 6 commits into from
Aug 12, 2019
Merged

CI: Move from AppVeyor to Azure Pipelines #783

merged 6 commits into from
Aug 12, 2019

Conversation

effigies
Copy link
Member

Starting by adapting scikit learn's configs and we'll see where things go. Azure seems to be much more than pipelines, so hopefully I can figure this out...

@effigies
Copy link
Member Author

@skoudoro In case you're looking for as simple a model for moving to Azure as possible... Also, I've registered the nipy org for Azure, so hopefully I can make sure you have access, too.

@codecov-io
Copy link

codecov-io commented Jul 30, 2019

Codecov Report

Merging #783 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #783      +/-   ##
==========================================
- Coverage   90.03%   89.97%   -0.07%     
==========================================
  Files          94       94              
  Lines       12018    12009       -9     
  Branches     2136     2133       -3     
==========================================
- Hits        10821    10805      -16     
- Misses        852      859       +7     
  Partials      345      345
Impacted Files Coverage Δ
nibabel/ecat.py 86.27% <0%> (-1.72%) ⬇️
nibabel/casting.py 86.44% <0%> (-0.8%) ⬇️
nibabel/streamlines/trk.py 94.31% <0%> (-0.12%) ⬇️
nibabel/affines.py 100% <0%> (ø) ⬆️
nibabel/cmdline/parrec2nii.py 32.73% <0%> (ø) ⬆️
nibabel/quaternions.py 99.02% <0%> (ø) ⬆️
nibabel/nifti1.py 91.48% <0%> (+0.01%) ⬆️
nibabel/nicom/dwiparams.py 71.69% <0%> (+0.54%) ⬆️
nibabel/__init__.py 93.75% <0%> (+1.02%) ⬆️

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 cee6bf3...9b0e4c9. Read the comment docs.

@effigies
Copy link
Member Author

Note that #665 is alive and well in here.

@matthew-brett
Copy link
Member

Nice to see that #665 is still keeping in shape ... :)

@effigies
Copy link
Member Author

I think its stochasticity is an effect of dict ordering, which is why it only shows up in Python < 3.6, so I guess if we just wait for one more end-of-life the tests will be fixed.

@effigies effigies merged commit 2f2c308 into master Aug 12, 2019
@effigies effigies deleted the ci/azure branch August 12, 2019 19:38
@skoudoro
Copy link
Member

Nice 👍 , Can you add me to the nipy org for Azure @effigies? I will add DIPY project this week or next one.

@effigies
Copy link
Member Author

@skoudoro I'll try to figure that out. Ping me again if I haven't in a couple days.

@effigies
Copy link
Member Author

@skoudoro I tried to add you to the nipy organization on Azure (also @satra, @matthew-brett and @arokem, because I could think of their emails without looking them up). It is not very intuitive.

Here are the docs: https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github?view=azure-devops&tabs=yaml#organizations

Let me know if that works out for you, or if you need me to try to bump up your permissions. I'm not sure how to make you all admins, but I made you "stakeholders", which might be roughly equivalent.

@effigies effigies modified the milestones: 3.0.0 RC1, 3.0.0 Oct 23, 2019
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.

4 participants