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

Remove suppression of files checked by cppcheck in action to allow visual confirmation it worked #1488

Closed
skliper opened this issue May 6, 2021 · 1 comment · Fixed by #1489 or #1492
Closed

Remove suppression of files checked by cppcheck in action to allow visual confirmation it worked #1488

skliper opened this issue May 6, 2021 · 1 comment · Fixed by #1489 or #1492
Assignees
@skliper
Labels
continuous-integration enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented May 6, 2021

Is your feature request related to a problem? Please describe.
The "all" static analysis check suppresses standard output with --quiet, so can't see that it actually did anything

Describe the solution you'd like
Remove --quiet

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 7.0.0 milestone May 6, 2021
@skliper skliper self-assigned this May 6, 2021
skliper added a commit to skliper/cFE that referenced this issue May 6, 2021
@skliper
Fix nasa#1488, Report files checked in cppcheck action
8ecaead
@skliper skliper mentioned this issue May 6, 2021
Merged
@astrogeco
Copy link
Contributor

Need to do the same in osal

@astrogeco astrogeco mentioned this issue May 11, 2021
Closed
astrogeco added a commit that referenced this issue May 11, 2021
@astrogeco
Merge pull request #1489 from skliper/fix1488-loud_cppcheck
6638ed3 
Fix #1488, Report files checked in cppcheck action
This was referenced May 11, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
continuous-integration enhancement
Projects
None yet
Milestone
Caelum
2 participants
@skliper @astrogeco