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

fix: edit protocgen script and generate common types #1311

Merged
merged 6 commits into from
Feb 1, 2023

Conversation

Vvaradinov
Copy link
Contributor

Description

The old protocgen.sh script was only targeting the x/ modules folder and wasn't generating the common types that are present on Ethermint. This PR modifies the script and generates the common types.


Closes #XXX

@Vvaradinov Vvaradinov requested a review from a team as a code owner February 1, 2023 08:49
@Vvaradinov Vvaradinov requested review from MalteHerrmann and GAtom22 and removed request for a team February 1, 2023 08:49
Copy link
Contributor

@MalteHerrmann MalteHerrmann left a comment

Choose a reason for hiding this comment

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

Good catch! Approved except for failing linter

CHANGELOG.md Outdated Show resolved Hide resolved
Vvaradinov and others added 3 commits February 1, 2023 10:56
Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Feb 1, 2023

Codecov Report

Merging #1311 (5120bef) into main (c0dba09) will decrease coverage by 0.02%.
The diff coverage is 70.40%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1311      +/-   ##
==========================================
- Coverage   72.42%   72.40%   -0.02%     
==========================================
  Files         177      178       +1     
  Lines       10351    10449      +98     
==========================================
+ Hits         7497     7566      +69     
- Misses       2623     2645      +22     
- Partials      231      238       +7     
Impacted Files Coverage Δ
crypto/ethsecp256k1/ethsecp256k1.go 70.40% <70.40%> (ø)

@Vvaradinov Vvaradinov merged commit 059cb2e into main Feb 1, 2023
@Vvaradinov Vvaradinov deleted the Vvaradinov/common-types branch February 1, 2023 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants