-
Notifications
You must be signed in to change notification settings - Fork 133
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
Always install ExamplePlugin on build_raven #1495
Conversation
Job Test CentOS 7 on 1d25842 : invalidated by @joshua-cogliati-inl restarted civet |
Is the "Refs. #1489" supposed to close the issue? If so, can you edit the description to close the issue? |
e0e4f44
to
8ce76f0
Compare
@alfoa Pretty much all the tests have passed on this if you want to give it another once over. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR APPROVED
PR approved. Checklist passed. No email is needed since this is a crash of the installation but not wrong results can be produced |
Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #1489
What are the significant changes in functionality due to this change request?
Adds a line to ./build_raven that will always install ExamplePlugin when the script is run.
For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.
<internalParallel>
to True.raven/tests/framework/user_guide
andraven/docs/workshop
) have been changed, the associated documentation must be reviewed and assured the text matches the example.