Skip to content
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

ENH: Update project name in CMake #23

Merged
merged 1 commit into from
Sep 14, 2021
Merged

ENH: Update project name in CMake #23

merged 1 commit into from
Sep 14, 2021

Conversation

tbirdso
Copy link
Contributor

@tbirdso tbirdso commented Aug 12, 2021

The project repository was renamed from Mesh3DProcrustesAlignTest to ITKShape shortly after creation, but CMake build generation continued to use the previous name. This PR fixes the project name in CMake in preparation for adding to ITK as a remote module per Issue 22.

@tbirdso tbirdso requested a review from thewtex August 12, 2021 17:17
@tbirdso
Copy link
Contributor Author

tbirdso commented Aug 16, 2021

@thewtex Could you please take a look at this PR as a sanity check? This is a small PR so that ITKShape will build as an ITK remote module under the name "ITKShape" rather than "ITKMesh3DProcrustesAlignFilter".

@tbirdso tbirdso requested a review from vicory August 30, 2021 15:42
@tbirdso
Copy link
Contributor Author

tbirdso commented Aug 30, 2021

Hi @vicory, as @thewtex is currently away could you please take a quick look over this PR if you have time?

@bpaniagua
Copy link
Contributor

bpaniagua commented Aug 30, 2021

Hi @allemangD can you please help out and review this PR and merge?

Hi @allemangD I did not know Jared was working on it. Please disregard.

@bpaniagua bpaniagua removed the request for review from allemangD August 30, 2021 16:31
@tbirdso
Copy link
Contributor Author

tbirdso commented Aug 30, 2021

Thanks @bpaniagua, whoever wants to review is fine, just trying to get another set of eyes on it before we add ITKShape as an ITK remote module.

CMakeLists.txt Outdated Show resolved Hide resolved
itk-module.cmake Outdated Show resolved Hide resolved
include/itkMeshProcrustesAlignFilter.h Outdated Show resolved Hide resolved
@tbirdso
Copy link
Contributor Author

tbirdso commented Sep 14, 2021

Fixed formatting with .clang-format. Can look into other module template updates in a future PR.

@tbirdso
Copy link
Contributor Author

tbirdso commented Sep 14, 2021

Going to go ahead and merge this as all tests passed prior to style fix.

Thanks for reviewing @dzenanz !

@tbirdso tbirdso merged commit 87faabf into main Sep 14, 2021
@thewtex thewtex deleted the rename-project branch December 19, 2022 14:52
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.

None yet

4 participants