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

Missing cppcheck in github actions workflow #166

Closed
astrogeco opened this issue Jan 4, 2021 · 0 comments · Fixed by #171 or #172
Closed

Missing cppcheck in github actions workflow #166

astrogeco opened this issue Jan 4, 2021 · 0 comments · Fixed by #171 or #172
Labels
bug continuous-integration
Milestone
Caelum

Comments

@astrogeco
Copy link
Contributor

Describe the bug

cppcheck test is no longer checked in continuous integration after migrating from TravisCI

Expected behavior
A step that runs cpp check before compiling

Additional context
None
@astrogeco astrogeco changed the title Add cppcheck to github actions workflow Missing cppcheck in github actions workflow Jan 4, 2021
@astrogeco astrogeco mentioned this issue Jan 4, 2021
Merged
astrogeco added a commit to astrogeco/cFS that referenced this issue Jan 4, 2021
@astrogeco
Fix nasa#166, Add static analysis to CI workflow
d061ead
astrogeco added a commit that referenced this issue Jan 5, 2021
@astrogeco
Merge pull request #171 from astrogeco/fix-167-and-169-cppcheck-and-p…
99d45e6 
…r-workflow

Fix #166 and #169, Add Static Code Analysis and Pull Request Trigger to CI workflow
@astrogeco astrogeco added this to the Caelum cFS-20.12 milestone Jan 5, 2021
@astrogeco astrogeco mentioned this issue Jan 6, 2021
Merged
chillfig pushed a commit to chillfig/cFS that referenced this issue Mar 17, 2022
@jphickey
Fix nasa#166, improve coverage on CF_CmdGetSetParam
e932449 
Rewrites the unit test on this function in order to get full line coverage.
This needs to call the routine with every defined param ID to get each case.
chillfig pushed a commit to chillfig/cFS that referenced this issue Mar 17, 2022
@astrogeco
Merge pull request nasa#181 from jphickey/fix-166-coverage-getset
dddc89f 
Fix nasa#166, improve coverage on CF_CmdGetSetParam
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug continuous-integration
Projects
None yet
Milestone
Caelum
1 participant
@astrogeco