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

Jump to bottom

Remove CodeQL Paths Ignore #2147

Closed
ArielSAdamsNASA opened this issue Sep 19, 2022 · 0 comments · Fixed by #2148 or #2161
Closed

Remove CodeQL Paths Ignore #2147

ArielSAdamsNASA opened this issue Sep 19, 2022 · 0 comments · Fixed by #2148 or #2161
Assignees
@ArielSAdamsNASA
Labels
continuous-integration draco-rc4
Milestone
Draco

Comments

@ArielSAdamsNASA
Copy link

ArielSAdamsNASA commented Sep 19, 2022 •
edited
Loading

Describe the bug
CodeQL throws two issues: Using paths-ignore can prevent Code Scanning annotating new alerts in your pull requests and on push.

To Reproduce
View the latest CodeQL workflow annotations.

Expected behavior
Same behavior, with no issues thrown in the workflow.

Additional context
When removed, documentation files will still be ignored without the issues thrown due to this PR: nasa/cFS#577

Reporter Info
Ariel Adams, MCSG Tech
@ArielSAdamsNASA ArielSAdamsNASA self-assigned this Sep 19, 2022
ArielSAdamsNASA pushed a commit to ArielSAdams/cFE that referenced this issue Sep 19, 2022
@arielswalker
Fix nasa#2147, Remove CodeQL Paths Ignore
ccd0623
@ArielSAdamsNASA ArielSAdamsNASA mentioned this issue Sep 19, 2022
Merged
2 tasks
dzbaker added a commit that referenced this issue Oct 3, 2022
@dzbaker
Merge pull request #2148 from ArielSAdams/fix-2147-remove-codeql-path…
d9eadbf 
…s-ignore

Fix #2147, Remove CodeQL Paths Ignore
@dzbaker dzbaker mentioned this issue Oct 3, 2022
Merged
@dmknutsen dmknutsen added this to the Draco milestone Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ArielSAdamsNASA ArielSAdamsNASA

Labels
continuous-integration draco-rc4
Projects
None yet
Milestone
Draco
2 participants
@dmknutsen @ArielSAdamsNASA