-
Notifications
You must be signed in to change notification settings - Fork 40
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
Main2.0 #467
base: master
Are you sure you want to change the base?
Main2.0 #467
Conversation
Thanks @BDoignies, I think you miss the PR #466 but we can keep this one.. |
Oh sorry... my bad... Haven't seen yours... Either is fine as long as we modify accordingly the corresponding worflow file in DGtal CMake2.0 PR. |
no problem ;) I also miss to mention you, not a big deal. There is perhaps also the contribution if you have to look (more light I think) |
We should run a diff on the two PR; or directly merge them together and fix conflicts if any. After a quick look, yours look a bit more complete in the variable it modifies; but I might be missing something. |
Or perhaps you can cherry-pick if you see some that you miss ;) btw the let me know if you plane to looks the related DGtalTools-contrib repository . |
Hey @kerautret, I updated my PR to also modify variable WITH_VISU3D_QGLVIEWER, which was the only one missing if I am not mistaken. I do not plan to look at DGtalTools-contrib for now. My primary focus is updating DGtal. DGtalTools was modified to make sure the GH-action run on DGtal side. There will be further updates in order to also make sure DGtalTools action also run. Upon realease of DGtal 2.0, I will make sure, that the tools the community built are up-to-date with DGtal. If you wan't to start a PR for this, please do not forget to mention me :) |
Perfect thanks a lot @BDoignies fine for me, I can do it no problem. |
At the moment, something sets the minimum standard to 14. I suspect this is in the file included by DGtal.
Thanks a lot for contributing to DGtalTools, before submitting your PR, please fill up the description and make sure that all checkboxes are checked.
PR Description
The goal of this PR is to match changes in the branch DGTal/main2.0 (and corresponding) PRs.
Checklist
cmake
mode (otherwise, Github Actions C.I. will fail).