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

Fix #1483, Add CI workflow to run cFE coverage tests #1590

Merged
merged 7 commits into from
Jun 10, 2021

Conversation

pepepr08
Copy link
Contributor

@pepepr08 pepepr08 commented May 28, 2021

Describe the contribution
Fix #1483
Run cFE coverage tests for every push and pull request. Use a checks-for-duplicates job to avoid duplicate runs of the CI Action.

Testing performed
Steps taken to test the contribution:

  1. Pushed to fork, CI action ran successfully: https://github.com/pepepr08/cFE/actions/runs/886331016
  2. Created a temporary branch and introduced an ES bug to force a workflow failure. Workflow failed successfully: https://github.com/pepepr08/cFE/actions/runs/886371053
  3. Reverted commit that introduced bug to verify that workflow run completed successfully.

Expected behavior changes
A new workflow is added when looking at the different nasa/cFE actions. The workflow will run for every push and pull request.

System(s) tested on

  • Tested on the Github servers where CI actions get to run.

Contributor Info - All information REQUIRED for consideration of pull request
Jose F. Martinez Pedraza/NASA GSFC

@pepepr08 pepepr08 added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label May 28, 2021
@pepepr08 pepepr08 removed the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label May 28, 2021
@pepepr08 pepepr08 marked this pull request as ready for review June 1, 2021 19:23
@pepepr08 pepepr08 added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jun 1, 2021
@astrogeco
Copy link
Contributor

CCB:2021-06-02 APPROVED with Changes

  • switch BUILDTYPE from "release" to "debug"

@astrogeco astrogeco added CCB:Approved Indicates code review and approval by community CCB continuous-integration and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Jun 2, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate June 10, 2021 21:25
@astrogeco astrogeco merged commit 569a646 into nasa:integration-candidate Jun 10, 2021
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates code review and approval by community CCB continuous-integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a CI action to build and execute the cFE coverage tests
3 participants