Skip to content

Conversation

@kaymckay
Copy link
Contributor

@kaymckay kaymckay commented Mar 7, 2022

Waiting for approval here to mass distribute

@kaymckay kaymckay requested review from a team March 7, 2022 15:34
@kaymckay kaymckay self-assigned this Mar 7, 2022
@kaymckay kaymckay marked this pull request as ready for review March 7, 2022 15:34
@codecov
Copy link

codecov bot commented Mar 7, 2022

Codecov Report

Merging #606 (06b3caf) into master (a1f558c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #606   +/-   ##
=======================================
  Coverage   54.34%   54.34%           
=======================================
  Files         181      181           
  Lines       15180    15180           
=======================================
  Hits         8250     8250           
  Misses       6610     6610           
  Partials      320      320           

@kaymckay kaymckay requested review from a team and ecrupper March 8, 2022 01:07
Copy link
Contributor

@jbrockopp jbrockopp left a comment

Choose a reason for hiding this comment

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

LGTM

@wass3r
Copy link
Collaborator

wass3r commented Mar 15, 2022

@ecrupper merge is blocked by your review. can you re-review and merge if it's good?

thanks @kaymckay 🥳

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.

5 participants