Skip to content

Conversation

@ecrupper
Copy link
Contributor

Continuing effort from these: #861, #882, #884, #913, #915, #928, #932

@ecrupper ecrupper self-assigned this Aug 22, 2023
@ecrupper ecrupper requested a review from a team as a code owner August 22, 2023 14:25
@codecov
Copy link

codecov bot commented Aug 22, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@a980c86). Click here to learn what that means.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #933   +/-   ##
=======================================
  Coverage        ?   70.89%           
=======================================
  Files           ?      310           
  Lines           ?    12743           
  Branches        ?        0           
=======================================
  Hits            ?     9034           
  Misses          ?     3255           
  Partials        ?      454           
Files Changed Coverage Δ
api/build/cancel.go 0.00% <0.00%> (ø)
api/build/clean.go 0.00% <0.00%> (ø)
database/step/create.go 85.00% <80.00%> (ø)
database/step/update.go 85.00% <80.00%> (ø)

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

Copy link
Contributor

@plyr4 plyr4 left a comment

Choose a reason for hiding this comment

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

nice

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