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

Add regression_tests to the ALL target. #1376

Merged
merged 2 commits into from
Dec 13, 2022

Conversation

andrew-platt
Copy link
Collaborator

Fixes issue #1325

Feature or improvement description
When BUILD_TESTING is enabled, a user would generally expect that a make install command would include everything for the regression testing. This PR adds the regression_tests target to the ALL target in the CMake build system.

Related issue, if one exists
#1325

Impacted areas of the software
This only affects the cmake build system when testing is enabled.

Copy link
Collaborator

@deslaughter deslaughter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding ALL to regression_test_module_drivers isn't necessary as it's a dependency of regression_tests, but it shouldn't hurt either. I think this is ready to be merged.

@andrew-platt andrew-platt merged commit 650b605 into OpenFAST:dev Dec 13, 2022
@andrew-platt andrew-platt deleted the b/regTest_ALL branch December 19, 2022 15:44
@rafmudaf rafmudaf mentioned this pull request Dec 19, 2022
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants