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 #55 #56, Remove action branch restrictions, add duplicate checks, fix format check job name, remove --quiet #57

Closed
wants to merge 3 commits into from

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Oct 22, 2021

Describe the contribution

Remove action branch restrictions, add duplicate checks, fix format check job name, remove --quiet

Testing performed
CI

Expected behavior changes

  • Runs on all push/pr
  • Skips on duplicate
  • Format check job name correct
  • Reports files analyzed by cppcheck

System(s) tested on
CI

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC
@astrogeco

astrogeco and others added 2 commits September 10, 2021 23:37
Enable format-check and static-analysis workflows to run on push to
arbitrary branches.

Enable CodeQL workflow to run on pull requests targeting arbitrary
branches.
@ArielSAdamsNASA
Copy link

Does the duplicate check job work without if: ${{ needs.check-for-duplicates.outputs.should_skip != 'true' }}?

@skliper
Copy link
Contributor Author

skliper commented Oct 22, 2021

@ArielSAdamsNASA Doesn't look like it, good catch! Fixing now...

@skliper skliper force-pushed the fix56-cppcheck_rm_quiet branch from 2be5d19 to f8f32a0 Compare October 22, 2021 17:15
@skliper skliper removed the CCB:Ready label Oct 27, 2021
@skliper
Copy link
Contributor Author

skliper commented Oct 28, 2021

OBE, #57 is a better solution.

@skliper skliper closed this Oct 28, 2021
@skliper skliper deleted the fix56-cppcheck_rm_quiet branch October 28, 2021 17:36
@skliper skliper added the duplicate This issue or pull request already exists label Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous-integration duplicate This issue or pull request already exists
Projects
None yet
3 participants