Skip to content

Re-add static code analysis for pull requests #740

@astrogeco

Description

@astrogeco

Describe the bug
The Travis CI runner ran cppcheck which we don't have anymore

To Reproduce
Submit pull request, notice that Travis CI doesn't run anymore.

Expected behavior
A success or failure report of a cppcheck run.

Code snips
Check from travis.yml

  #cppcheck flight software osal/src/bsp, osal/src/os
    - cppcheck --force --inline-suppr --std=c99 --language=c --error-exitcode=1
      --enable=warning,performance,portability,style --suppress=variableScope
      --inconclusive --quiet src/bsp src/os 2>cppcheck_flight_osal.txt
    - |
      if [[ -s cppcheck_flight_osal.txt ]]; then
        echo "You must fix cppcheck errors before submitting a pull request"
        echo ""
        cat cppcheck_flight_osal.txt
        exit -1
      fi

System observed on:
n/a

Additional context
Part of ongoing TravisCI to GitHub Actions migration

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions