ENH: Add Azure Pipelines CI. #58
Conversation
|
The target and image within the project template will surely need some revision. And so will need the Updating the ITK tag to ITK v5 will need to be done soon ! |
thewtex
left a comment
thewtex
left a comment
There was a problem hiding this comment.
Thanks @jhlegarreta !
A few comments inline.
I disabled CircleCI, TravisCI, and Appveyor. on this repository.
| command: | | ||
| curl -L https://raw.githubusercontent.com/InsightSoftwareConsortium/ITK/dashboard/itk_common.cmake -O | ||
| - run: | ||
| name: Evaluate template |
There was a problem hiding this comment.
This step is needed in the new azure-pipelines.yml
There was a problem hiding this comment.
I do not see the missing step in the added file. The CTest driver script is being fetched actually. Where do we need to add it ? Does this apply to all azure-pipelines.yml files? The ITKSplitComponents azure-pipelines.yml does not include it 🤔 .
There was a problem hiding this comment.
Only the ITKModuleTemplate repository, we need to evaluate the template with the defaults,
python -m cookiecutter --no-input /ITKModuleTemplateCookieCutter
Compare ITKModuleTemplate/.circleci/config.yml with ITKModuleTemplate/{{cookiecutter.project_name}}/.circleci/config.yml.
There was a problem hiding this comment.
Thanks for the explanation 👍 . Let's see if I did this correctly in c267ab4.
jhlegarreta
force-pushed
the
UseAzurePipelinesForCI
branch
from
9c0c91b to
c267ab4
Compare
Add Azure Pipelines CI, which supersedes Appveyor, Circle and Travis CI. Update the README file badges accordingly.
jhlegarreta
force-pushed
the
UseAzurePipelinesForCI
branch
from
c267ab4 to
a587ac1
Compare
|
@thewtex thanks 👍 |
thewtex
left a comment
thewtex
left a comment
There was a problem hiding this comment.
Thanks @jhlegarreta !
Let's merge, then I will enable the builds, then we may need to follow up with fixes.
2 participants
Add Azure Pipelines CI, which supersedes Appveyor, Circle and Travis CI.
Update the
READMEfile badges accordingly.