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.

Sign up for GitHub

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

Jump to bottom

Display cppcheck results in GitHub code scanning dashboard #416

Closed
3 tasks done
astrogeco opened this issue Jan 27, 2022 · 0 comments · Fixed by #514
Closed
3 tasks done

Display cppcheck results in GitHub code scanning dashboard #416

astrogeco opened this issue Jan 27, 2022 · 0 comments · Fixed by #514
Assignees
@ArielSAdamsNASA
Labels
continuous-integration good first issue Good for newcomers

Comments

@astrogeco
Copy link
Contributor

astrogeco commented Jan 27, 2022 •
edited
Loading

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I reviewed the cFS README.md file to see if the feature is in the major future work.
  • I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
Convert cppcheck's plist output to SARIF using https://github.com/sarif-standard/sarif-sdk-typescript

Use the codeql-upload action to upload SARIF file to CodeScanning

Describe alternatives you've considered
None

Additional context
None

Requester Info
Gerardo E. Cruz-Ortiz, NASA/GSFC
@astrogeco astrogeco added the good first issue Good for newcomers label Apr 19, 2022
ArielSAdamsNASA pushed a commit to ArielSAdams/cFS that referenced this issue Jun 21, 2022
@arielswalker
Fix nasa#416, Display cppcheck in Dashboard
ad45903
ArielSAdamsNASA pushed a commit to ArielSAdams/cFS that referenced this issue Jun 21, 2022
@arielswalker
Fix nasa#416, Display cppcheck in Dashboard
e56377e
@ArielSAdamsNASA ArielSAdamsNASA mentioned this issue Jun 21, 2022
Merged
2 tasks
dzbaker added a commit that referenced this issue Oct 11, 2022
@dzbaker
Merge pull request #514 from ArielSAdams/fix-416-display-cppcheck-das…
58a58c4 
…hboard

Fix #416, Display cppcheck in Code Scanning Alerts Dashboard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ArielSAdamsNASA ArielSAdamsNASA

Labels
continuous-integration good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #416, Display cppcheck in Code Scanning Alerts Dashboard ArielSAdams/cFS
2 participants
@astrogeco @ArielSAdamsNASA