-
Notifications
You must be signed in to change notification settings - Fork 116
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
Create check_methoddoc_gen.yaml #19879
Conversation
Create GitHub action for checking MethodDoc.C is generated when functions.rst is changed.
@JustinPrivitera and @cyrush this adds CI for keeping It would be nice to cancel the |
BTW, you can introspect workflow runs by going to Actions tab. |
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.
I am guessing that the fix to methoddoc and functions was how you tested this?
Yes. Thats right. Along with various commits of whitespace (ws) changes to test different branches of the logic. |
@cyrush any qualms about merging this to |
This should wait until we decide upon #19881 |
I think we would want this both on RC and develop if possible. |
Well, now I am inclined to wait until we resolve the email discussion...do we really need to keep Thoughts? |
Description
Create GitHub action for checking MethodDoc.C is generated when functions.rst is changed.
Type of change
[ ] Bug fix[ ] Documentation update[ ] OtherHow Has This Been Tested?
Reminders:
Checklist:
[ ] I have updated the release notes.[ ] I have added debugging support to my changes.[ ] I have added tests that prove my fix is effective or that my feature works.[ ] I have confirmed new and existing unit tests pass locally with my changes.[ ] I have added new baselines for any new tests to the repo.