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

cFS Bundle Integration Candidate: Caelum+dev3 #414

Merged
merged 4 commits into from
Feb 3, 2022
Merged

cFS Bundle Integration Candidate: Caelum+dev3 #414

merged 4 commits into from
Feb 3, 2022

Commits on Feb 3, 2022

  1. Fix #412, Rename CodeQL workflows 

    Renames workflows to better describe what each one does. The CodeQL
"reusable" workflow is meant to be used by other workflows.

The CodeQL "Analysis" workflow calls the "reusable" CodeQL workflow to
execute the static analysis runs.

Co-authored-by: Ariel Adams   <ArielSAdamsNASA@users.noreply.github.com>
    @astrogeco @ArielSAdamsNASA
    astrogeco and ArielSAdamsNASA committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    1d80995 View commit details
    Browse the repository at this point in the history

  2. Fix #412, resolve error in CodeQL Analyze Action 

    Fixes errors in CodeQL results uploads step.

Update parameters in CodeQL "reusable" workflow.

BREAKING Interface changes:

- Renames callable workflow to `codeql-reusable.yml`, submodules will
have to be updated
- Adds required `component-path` input parameter
- Repurpose tests input to be a boolean tied to "ENABLE_UNIT_TESTS" flag

Internal changes:

- Use git clone instead of checkout@v2 for the cFS-Bundle
- Use symlink to map calling repo workspace to expected cFS Bundle directory location

- Enable "code snippets" option to CodeQL Analyze action
- Archives sarif files from analysis output
- Removes code duplication by using a matrix build for security and coding standard analyses
- Alphabetizes workflow inputs and order based on "required" flag
    @astrogeco
    astrogeco committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    98e2ef8 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #413 from astrogeco/fix-codeql-workflow 

    Fix #412, git error in CodeQL Analyze Action
    @astrogeco
    astrogeco committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    d6bca6e View commit details
    Browse the repository at this point in the history

  4. cFS Integration Candidate: Caelum+dev3 

    *cFE v7.0.0-rc4+dev70*

nasa/cFE#2041, Improve CFE_SB_IsValidMsgId handler
nasa/cFE#2034, Update CodeQL workflow
nasa/cFE#2042, Replace CFE_SB_ValueToMsgId(0) with CFE_SB_INVALID_MSG_ID

*osal v6.0.0-rc4+dev32*

nasa/osal#1209, typesafe definition of osal_id_t

*sample_app v1.3.0-rc4+dev9*

nasa/sample_app#165, Use preferred UT patterns

Co-authored-by: Jacob Hageman   <skliper@users.noreply.github.com>
Co-authored-by: Paul            <paulober@users.noreply.github.com>
Co-authored-by: Ariel Adams     <ArielSAdamsNASA@users.noreply.github.com>
    @astrogeco @skliper
    @paulober @ArielSAdamsNASA
    4 people committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    267cb8a View commit details
    Browse the repository at this point in the history