-
Notifications
You must be signed in to change notification settings - Fork 121
pmd into Workflow
#882
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
pmd into Workflow
#882
Conversation
|
Yeah I can imagine that being the case. That's no problem, we have much messier GH action workflows. I'll review this soon. |
|
The PMD Analysis failed due to mainly what I suspect the need to set environment variable |
Co-authored-by: Spencer Bryngelson <[email protected]>
Description
(#646), I am attempting to implement PMD GitHub Action to the Test Suite.
Fixes #(issue) [optional]
None
Please delete options that are not relevant.
Scope
If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration
Test Configuration:
Checklist
docs/)examples/that demonstrate my new feature performing as expected.They run to completion and demonstrate "interesting physics"
./mfc.sh formatbefore committing my codeIf your code changes any code source files (anything in
src/simulation)To make sure the code is performing as expected on GPU devices, I have:
nvtxranges so that they can be identified in profiles./mfc.sh run XXXX --gpu -t simulation --nsys, and have attached the output file (.nsys-rep) and plain text results to this PR./mfc.sh run XXXX --gpu -t simulation --rsys --hip-trace, and have attached the output file and plain text results to this PR.