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

refactor: removing the upgrade handlers #399

Merged
merged 3 commits into from
May 31, 2023

Conversation

spoo-bar
Copy link
Contributor

Cleaning up the upgrade handlers prepping for v1

@spoo-bar spoo-bar marked this pull request as ready for review May 31, 2023 10:29
@spoo-bar spoo-bar requested a review from a team as a code owner May 31, 2023 10:30
@spoo-bar spoo-bar requested a review from fdymylja May 31, 2023 10:30
@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.18 ⚠️

Comparison is base (c1bbbd4) 69.34% compared to head (c0f3fad) 69.17%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #399      +/-   ##
==========================================
- Coverage   69.34%   69.17%   -0.18%     
==========================================
  Files          67       67              
  Lines        3980     3980              
==========================================
- Hits         2760     2753       -7     
- Misses       1124     1130       +6     
- Partials       96       97       +1     
Impacted Files Coverage Δ
app/app_upgrades.go 28.57% <ø> (-33.34%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@zanicar zanicar merged commit dd58dde into main May 31, 2023
6 checks passed
@zanicar zanicar deleted the spoorthi/cleanup-upgrade-handlers branch May 31, 2023 13:11
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.

None yet

3 participants