-
Notifications
You must be signed in to change notification settings - Fork 62
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
Workflows #218
Conversation
… imported, it does seem necessary)
Merging this PR. The build error with MFEM master will be address in next PR. |
Well. I noticed it is still in draft mode. @justinlaughlin, please confirm that it is ready to review and merge. |
It is ready to merge - does it still say draft mode? I'll add in the option for triggering off of base MFEM in a separate branch. |
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.
Thank you for the improvement. I will merge this. I suppose that some test with master will still fail because ex18/ex18p were updated. Otherwise, it should go through. Let's see.
Do not merge
I'm trying to test a workflow file but according to what I've read on stack exchange, github needs a pull request to register the workflow before it can be triggered manually.
https://stackoverflow.com/questions/63362126/github-actions-how-to-run-a-workflow-created-on-a-non-master-branch-from-the-wo