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

imp: remove crisis module in v15 upgrade #1847

Merged
merged 5 commits into from
Oct 12, 2023
Merged

Conversation

facs95
Copy link
Contributor

@facs95 facs95 commented Oct 11, 2023

Description

Removing crisis module since it is not being used an instantiating v15 upgrade handler.


Closes #XXX

@facs95 facs95 requested a review from a team as a code owner October 11, 2023 22:19
@facs95 facs95 requested review from Vvaradinov and MalteHerrmann and removed request for a team October 11, 2023 22:19
@facs95 facs95 changed the title imp: Remove crisis module in v15 upgrade imp: Remove crisis module in v15 upgrade Oct 11, 2023
@facs95 facs95 changed the title imp: Remove crisis module in v15 upgrade imp: remove crisis module in v15 upgrade Oct 11, 2023
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #1847 (6fac846) into main (57b7d64) will decrease coverage by 0.03%.
Report is 8 commits behind head on main.
The diff coverage is 16.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1847      +/-   ##
==========================================
- Coverage   70.45%   70.42%   -0.03%     
==========================================
  Files         313      313              
  Lines       23324    23310      -14     
==========================================
- Hits        16432    16416      -16     
- Misses       6070     6072       +2     
  Partials      822      822              
Files Coverage Δ
app/export.go 13.01% <ø> (+0.43%) ⬆️
app/app.go 81.87% <16.66%> (-0.75%) ⬇️

... and 1 file with indirect coverage changes

Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

Looks great!

@fedekunze fedekunze merged commit a456910 into main Oct 12, 2023
31 of 33 checks passed
@fedekunze fedekunze deleted the facs95/remove-crisis-module branch October 12, 2023 06:11
@luchenqun
Copy link
Contributor

What are the reasons for removing this Crisis module?

@GAtom22
Copy link
Contributor

GAtom22 commented Oct 16, 2023

https://github.com/Mergifyio backport release/v15.0.x

@mergify
Copy link
Contributor

mergify bot commented Oct 16, 2023

backport release/v15.0.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Oct 16, 2023
(cherry picked from commit a456910)

# Conflicts:
#	CHANGELOG.md
#	app/app.go
#	app/upgrades/v15/constants.go
#	app/upgrades/v15/upgrades.go
GAtom22 added a commit that referenced this pull request Oct 16, 2023
* imp: remove `crisis` module in v15 upgrade (#1847)

(cherry picked from commit a456910)

# Conflicts:
#	CHANGELOG.md
#	app/app.go
#	app/upgrades/v15/constants.go
#	app/upgrades/v15/upgrades.go

* fix conflicts

---------

Co-authored-by: Freddy Caceres <facs95@gmail.com>
Co-authored-by: tom <tomasguerraalda@hotmail.com>
Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com>
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

4 participants