kerautret commented 1 year ago

PR Description

As done for DGtal, it also integrate GitAction configuration for windows architecture and removes appveyor check.

Checklist

[x] Doxygen documentation of the code completed (classes, methods, types, members...).
[x] Main tool doxygen documentation (following existing documentation of DGtalTools documentation.
[x] Check if it follows the tools structure described in CONTRIBUTING.md
[x] New entry in the ChangeLog.md added.
[x] Update the readme with potentially a screenshot of the tools if it applies.
[x] No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
[x] All continuous integration tests pass (Github Actions & appveyor).

kerautret commented 1 year ago

@dcoeurjo looks ready for review now ;) By the way could you remove the appveyor trigger on the settings ?

kerautret commented 1 year ago

ping @dcoeurjo if you have time to remove appveyor in the setting ;)

dcoeurjo commented 1 year ago

I think I've removed appveyor...

dcoeurjo commented 1 year ago

Seems to still be there... maybe just remove the appveryor.yaml

kerautret commented 1 year ago

I think I've removed appveyor...

thanks,

Seems to still be there... maybe just remove the appveryor.yaml

yes but it is well removed in the PR, perhaps I have to make a new commit (but needs conan fix of DGTal PR)

kerautret commented 1 year ago

Thanks @dcoeurjo looks fine now ;)