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

feat(app): replace staking with dogfood #42

Merged
merged 27 commits into from
Apr 29, 2024

Conversation

MaxMustermann2
Copy link
Contributor

@MaxMustermann2 MaxMustermann2 commented Apr 23, 2024

The dogfood module is designed to act as a drop-in replacement for the staking module. This PR enables that usage, and at the same time, helps rationalize parts of app/app.go by providing comments about each module and their orders (begin / end / genesis) within the overall system.

TODO: update the following things to remove reliance on staking module.

  • local_node.sh
  • exocored testnet init-files
  • app/ethtest_helper.go
  • app/test_helpers.go

TODO: testing of cross-chain communication.

@cloud8little
Copy link
Contributor

Test the commit af27899 with bootstrap test, passed.

image

@github-actions github-actions bot added the C:CLI label Apr 23, 2024
ret := am.keeper.EndBlock(ctx)
am.keeper.TrackHistoricalInfo(ctx)
return ret
return am.keeper.EndBlock(ctx)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning

path flow from Begin/EndBlock to a panic call
var ithPeers []string
for j := 0; j < args.numValidators; j++ {
if i != j {
ip, _ := getIP(j, args.startingIPAddress)

Check warning

Code scanning / gosec

Returned error is not propagated up the stack. Warning test

Returned error is not propagated up the stack.

// x/operator initialization - address only
privkey, _ := ethsecp256k1.GenerateKey()
key, _ := privkey.ToECDSA()

Check warning

Code scanning / gosec

Returned error is not propagated up the stack. Warning test

Returned error is not propagated up the stack.

// x/operator initialization - address only
privkey, _ := ethsecp256k1.GenerateKey()

Check warning

Code scanning / gosec

Returned error is not propagated up the stack. Warning test

Returned error is not propagated up the stack.
bondAmt := sdk.DefaultPowerReduction
// x/operator initialization - address only
privkey, _ := ethsecp256k1.GenerateKey()
key, _ := privkey.ToECDSA()

Check warning

Code scanning / gosec

Returned error is not propagated up the stack. Warning

Returned error is not propagated up the stack.

bondAmt := sdk.DefaultPowerReduction
// x/operator initialization - address only
privkey, _ := ethsecp256k1.GenerateKey()

Check warning

Code scanning / gosec

Returned error is not propagated up the stack. Warning

Returned error is not propagated up the stack.
@MaxMustermann2 MaxMustermann2 marked this pull request as ready for review April 29, 2024 09:19
@MaxMustermann2 MaxMustermann2 merged commit 76a739d into imua-xyz:develop Apr 29, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants