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

chore(cmd/gossamer): Request chain spec as an argument in import-runtime command #3067

Merged

Conversation

EclesioMeloJunior
Copy link
Member

Changes

  • import-runtime argument was tightly coupled with the gssmr genesis spec file so in order to remove this dependency I've introduced the genesis spec file as an argument to the import-runtime command

Tests

go test -tags integration github.com/ChainSafe/gossamer/cmd/gossamer

Issues

Primary Reviewer

@jimjbrettj

Copy link
Contributor

@jimjbrettj jimjbrettj left a comment

Choose a reason for hiding this comment

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

Love to see it!

@codecov
Copy link

codecov bot commented Jan 24, 2023

Codecov Report

Merging #3067 (e0d1256) into development (c958720) will increase coverage by 0.29%.
The diff coverage is 20.00%.

❗ Current head e0d1256 differs from pull request most recent head 183730d. Consider uploading reports for the commit 183730d to get more accurate results

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #3067      +/-   ##
===============================================
+ Coverage        51.65%   51.94%   +0.29%     
===============================================
  Files              220      220              
  Lines            28086    28081       -5     
===============================================
+ Hits             14507    14587      +80     
+ Misses           12289    12199      -90     
- Partials          1290     1295       +5     

Copy link
Contributor

@qdm12 qdm12 left a comment

Choose a reason for hiding this comment

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

Just a minor thing to change 😉

cmd/gossamer/main.go Show resolved Hide resolved
@EclesioMeloJunior EclesioMeloJunior merged commit de26200 into development Jan 30, 2023
@EclesioMeloJunior EclesioMeloJunior deleted the eclesio/remove-import-runtime-gssmr-dependency branch January 30, 2023 15:24
Copy link

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

remove the call to GetGssmrGenesisPath from createGenesisWithRuntime
3 participants