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.

Sign up for GitHub

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

Jump to bottom

chore: add default memIAVL config #1893

Merged
merged 8 commits into from
Oct 16, 2023
Merged

chore: add default memIAVL config #1893

merged 8 commits into from
Oct 16, 2023

Conversation

GAtom22
Copy link
Contributor

@GAtom22 GAtom22 commented Oct 16, 2023

Description

Closes #XXX

@codecov
Copy link

codecov bot commented Oct 16, 2023 •
edited
Loading

Codecov Report

Merging #1893 (178e080) into main (8a359af) will increase coverage by 0.00%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

❗ Current head 178e080 differs from pull request most recent head 27b3c72. Consider uploading reports for the commit 27b3c72 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1893   +/-   ##
=======================================
  Coverage   70.32%   70.33%           
=======================================
  Files         319      319           
  Lines       23554    23561    +7     
=======================================
+ Hits        16564    16571    +7     
  Misses       6170     6170           
  Partials      820      820
Files Coverage Δ
server/config/config.go 36.80% <100.00%> (+2.83%) ⬆️

@GAtom22 GAtom22 marked this pull request as ready for review October 16, 2023 20:46
@GAtom22 GAtom22 requested a review from a team as a code owner October 16, 2023 20:46
@GAtom22 GAtom22 requested review from Vvaradinov and 0xstepit and removed request for a team October 16, 2023 20:46
@GAtom22
Copy link
Contributor Author

GAtom22 commented Oct 16, 2023

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

@mergify
Copy link
Contributor

mergify bot commented Oct 16, 2023 •
edited
Loading

backport release/v15.0.x

✅ Backports have been created

@GAtom22 GAtom22 requested a review from facs95 October 16, 2023 20:47
facs95
facs95 approved these changes Oct 16, 2023
View reviewed changes
server/config/config.go Outdated Show resolved Hide resolved
@GAtom22 GAtom22 enabled auto-merge (squash) October 16, 2023 21:31
@GAtom22 GAtom22 merged commit 3e78845 into main Oct 16, 2023
24 of 25 checks passed
@GAtom22 GAtom22 deleted the GAtom22/memiavl-config branch October 16, 2023 21:44
mergify bot pushed a commit that referenced this pull request Oct 16, 2023
@GAtom22 @mergify
chore: add default memIAVL config (#1893)
d2d4964 
* chore: add default memIAVL config

* add constants

* add changelog entry

* fix lint issues

* add constants

* fix style

(cherry picked from commit 3e78845)

# Conflicts:
#	CHANGELOG.md
@mergify mergify bot mentioned this pull request Oct 16, 2023
Merged
GAtom22 added a commit that referenced this pull request Oct 16, 2023
@mergify @GAtom22
chore: add default memIAVL config (backport #1893) (#1897)
749411a 
* chore: add default memIAVL config (#1893)

* chore: add default memIAVL config

* add constants

* add changelog entry

* fix lint issues

* add constants

* fix style

(cherry picked from commit 3e78845)

# Conflicts:
#	CHANGELOG.md

* fix conflicts

---------

Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com>
Co-authored-by: tom <tomasguerraalda@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@facs95 facs95 facs95 approved these changes

@Vvaradinov Vvaradinov Awaiting requested review from Vvaradinov Vvaradinov is a code owner automatically assigned from evmos/core-engineering

@0xstepit 0xstepit Awaiting requested review from 0xstepit 0xstepit is a code owner automatically assigned from evmos/core-engineering

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@GAtom22 @facs95