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.

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

Rename CodeQL cFE Build and Add Duplicate Jobs #1035

Closed
ArielSAdamsNASA opened this issue May 20, 2021 · 0 comments · Fixed by #1039 or #1058
Closed

Rename CodeQL cFE Build and Add Duplicate Jobs #1035

ArielSAdamsNASA opened this issue May 20, 2021 · 0 comments · Fixed by #1039 or #1058
Assignees
Milestone

Comments

@ArielSAdamsNASA
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Users may be confused of the two CodeQL workflows. Multiple jobs may run at the same time.

Describe the solution you'd like
Make it clear that there is an analysis for osal using the cFE configuration build. Rename codeql-build.yml to codeql-cfe-build.yml. Rename the workflow from "CodeQL Analysis" to "CodeQL Analysis cFE Build".

cp ./cfe/cmake/Makefile.sample Makefile         
cp -r ./cfe/cmake/sample_defs sample_defs

Add the multiple jobs action so the workflow does not run multiple times.

Additional context
https://github.com/fkirc/skip-duplicate-actions

Requester Info
Ariel Adams, ASRC Federal

@ArielSAdamsNASA ArielSAdamsNASA self-assigned this May 20, 2021
astrogeco added a commit that referenced this issue Jun 2, 2021
…licate

Fix #1035, Rename CodeQL cFE Build and add Duplicate Job
pepepr08 pushed a commit to pepepr08/osal that referenced this issue Jun 9, 2021
@skliper skliper added this to the 6.0.0 milestone Sep 24, 2021
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
Fix 1034, Atomic type for SystemState
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants