close
Skip to content

Updating .github workflow for psalm-security-analysis based on outpu…#643

Merged
bobstrecansky merged 1 commit into
open-telemetry:mainfrom
bobstrecansky:update_psalm_security_analysis_action_workflow
Mar 30, 2022
Merged

Updating .github workflow for psalm-security-analysis based on outpu…#643
bobstrecansky merged 1 commit into
open-telemetry:mainfrom
bobstrecansky:update_psalm_security_analysis_action_workflow

Conversation

@bobstrecansky
Copy link
Copy Markdown
Contributor

#640 failed a psalm security check.
this action run mentions that we should avoid using on:push and should use on:pull_request.

This PR attempts to do just that.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 30, 2022

Codecov Report

Merging #643 (7cc7b0c) into main (e62cd5e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #643   +/-   ##
=========================================
  Coverage     84.45%   84.45%           
  Complexity     1141     1141           
=========================================
  Files           127      127           
  Lines          2767     2767           
=========================================
  Hits           2337     2337           
  Misses          430      430           
Flag Coverage Δ
7.4 84.40% <ø> (ø)
8.0 84.45% <ø> (ø)
8.1 84.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e62cd5e...7cc7b0c. Read the comment docs.

@bobstrecansky bobstrecansky merged commit 8f0b522 into open-telemetry:main Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants