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

Update codeql analysis frequency #4542

Merged
merged 1 commit into from
Sep 21, 2023

Conversation

MadVikingGod
Copy link
Contributor

Update CodeQL analysis to run on pushes to main, and PR to follow the recommendation of the Security Sig #4459.

@MadVikingGod MadVikingGod added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 21, 2023
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #4542 (f9162a2) into main (fd8eff8) will decrease coverage by 0.1%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #4542     +/-   ##
=======================================
- Coverage   81.3%   81.3%   -0.1%     
=======================================
  Files        220     220             
  Lines      17679   17679             
=======================================
- Hits       14386   14384      -2     
- Misses      2992    2994      +2     
  Partials     301     301             

@pellared
Copy link
Member

Update CodeQL analysis to run on pushes to main, and PR to follow the recommendation of the Security Sig #4459.

We need to do the same for https://github.com/open-telemetry/opentelemetry-go-contrib/blob/main/.github/workflows/codeql_analysis.yml

@MrAlias
Copy link
Contributor

MrAlias commented Sep 21, 2023

Merging prior to 1 day to include this in the evaluation of other PRs and since it is a trivial change.

@MrAlias MrAlias merged commit 206ed55 into open-telemetry:main Sep 21, 2023
@MrAlias MrAlias added this to the v1.19.0/v0.42.0 milestone Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants