Skip to content
This repository was archived by the owner on Mar 3, 2026. It is now read-only.

5% min starting point#1

Closed
lydia-pierce wants to merge 8 commits into
mainfrom
validator-minimim-5%
Closed

5% min starting point#1
lydia-pierce wants to merge 8 commits into
mainfrom
validator-minimim-5%

Conversation

@lydia-pierce

Copy link
Copy Markdown
Collaborator

Needs to be debugged, but it's a starting point modeled after Juno ChihuahuaChain#81

@evan-forbes evan-forbes left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We still need to add some more testing, and let's also investigate if #103 is also needed.

This is a perfect start tho, great work!

Comment thread app/app.go Outdated
Comment thread app/app.go Outdated
Comment thread app/app.go Outdated
lydia-pierce and others added 2 commits December 29, 2021 12:46
Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
Comment thread app/app.go
const (
AccountAddressPrefix = "chihuahua"
Name = "chihuahua"
upgradeName = ""

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

In #81 they named this "moneta-alpha" but ours doesn't have a name, so I left it blank.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

where is this used? Just to note that we will likely want to point this out when publishing our PR

@lydia-pierce

Copy link
Copy Markdown
Collaborator Author

what part of testutil/keeper/chihuahua.go did you comment out? The entire file?

@evan-forbes

Copy link
Copy Markdown

what part of testutil/keeper/chihuahua.go did you comment out? The entire file?

Yeah, the entire thing

@evan-forbes

evan-forbes commented Dec 29, 2021

Copy link
Copy Markdown

Let's also go ahead and add a simple changelog to indicate which changes have been included in this PR
example raw changelog markdown - example rendered changelog

lydia.pierce@principia.edu and others added 4 commits December 30, 2021 10:23
* add ability to add genesis accounts to the network config for integration testing

* add the keyring signer

* use the standard sdk way of integration testing

* delete unused file

* delete unused code
@evan-forbes

Copy link
Copy Markdown

closing as we went with a different solution

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants