Skip to content

Conversation

@jbrockopp
Copy link
Contributor

@jbrockopp jbrockopp added the bug Indicates a bug label Mar 11, 2022
@jbrockopp jbrockopp requested a review from a team as a code owner March 11, 2022 21:32
@jbrockopp jbrockopp self-assigned this Mar 11, 2022
@jbrockopp jbrockopp marked this pull request as draft March 11, 2022 21:32
@codecov
Copy link

codecov bot commented Mar 11, 2022

Codecov Report

Merging #341 (ebf547c) into master (695e86b) will increase coverage by 0.07%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #341      +/-   ##
==========================================
+ Coverage   85.25%   85.33%   +0.07%     
==========================================
  Files         119      119              
  Lines        5269     5215      -54     
==========================================
- Hits         4492     4450      -42     
+ Misses        637      629       -8     
+ Partials      140      136       -4     
Impacted Files Coverage Δ
action/pipeline/validate.go 92.68% <100.00%> (+4.54%) ⬆️

@jbrockopp jbrockopp marked this pull request as ready for review March 16, 2022 20:19
Copy link
Contributor

@KellyMerrick KellyMerrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@KellyMerrick KellyMerrick merged commit 6adbf1b into master Mar 16, 2022
@KellyMerrick KellyMerrick deleted the fix/compile/local branch March 16, 2022 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Indicates a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants