search

InsightSoftwareConsortium / ITKThickness3D

An ITK module to compute 3D thickness
Apache License 2.0
39 stars 18 forks source link

ENH: Remove unused CI config files #17

Closed jhlegarreta closed 5 years ago

jhlegarreta commented 5 years ago

Remove unused CI config files.

Appveyor, Travis, and Circle were superseded by Azure Pipelines CI in 29dd1d0.

jhlegarreta commented 5 years ago

@thewtex Shouldn't the azure-pipelines.yml be placed in the root instead of the test folder?

jhlegarreta commented 5 years ago

@T4mmi would you disable the Appveyor, Circle and Travis builds?

thewtex commented 5 years ago

@thewtex Shouldn't the azure-pipelines.yml be placed in the root instead of the test folder?

I think we should place the configuration in the test folder to avoid top level clutter. Some existing modules still need to be updated for this.

@T4mmi would you disable the Appveyor, Circle and Travis builds?

:+1: Done

T4mmi commented 5 years ago

@thewtex this means that CI build file should be placed in test folder for every ITK modules ? (or just while the transition from Travis/Circle/appveyor to Azure pipes ?)

thewtex commented 5 years ago

Yes, we should put the CI build file in the test folder for all ITK modules. Azure Pipelines make it optional where the configuration file is located, but that flexibility is not available in other services.