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

Disable git version checking when project is not top level #2909

Closed
wants to merge 2 commits into from

Conversation

daljit46
Copy link
Member

When building the project via add_subdirectory (or other means) as part of an external project, checking that the git tag of the project matches the version of MRtrix3 doesn't really make sense. This PR disables this check in that scenario to allow building external projects.

@daljit46 daljit46 added the build label May 16, 2024
@daljit46 daljit46 self-assigned this May 16, 2024
Copy link

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@daljit46 daljit46 force-pushed the fix_version_external_projects branch from 4acc524 to 5e70485 Compare May 16, 2024 13:28
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@daljit46 daljit46 marked this pull request as draft May 16, 2024 14:28
@daljit46 daljit46 force-pushed the fix_version_external_projects branch from 5e70485 to 53b38ec Compare June 6, 2024 09:27
Copy link

github-actions bot commented Jun 6, 2024

clang-tidy review says "All clean, LGTM! 👍"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant