Skip to content

refactor: refactor workspace and chainConfig class#88

Open
ulbqb wants to merge 4 commits into
spoo-bar:mainfrom
Finschia:fix/refactor
Open

refactor: refactor workspace and chainConfig class#88
ulbqb wants to merge 4 commits into
spoo-bar:mainfrom
Finschia:fix/refactor

Conversation

@ulbqb

@ulbqb ulbqb commented Dec 27, 2023

Copy link
Copy Markdown
Contributor

This PR changes following:

  • refactor workspace class
  • add construct function of chain config
  • use validate function check the chain config
  • set default value of signType and coinType
  • fix @cosmjs/crypto version

170210 and others added 4 commits December 27, 2023 12:40
* refactor: refactor workspace and chainConfig class

Signed-off-by: 170210 <j170210@icloud.com>

* refactor: validate and set default value

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix sync beaker.toml

Signed-off-by: 170210 <j170210@icloud.com>

* style: fix style

Co-authored-by: Shogo Hyodo <mmoshg8u@gmail.com>

* style: add blank line

Co-authored-by: Shogo Hyodo <mmoshg8u@gmail.com>

* fixup: move constants

Signed-off-by: 170210 <j170210@icloud.com>

* style: add blank line

Signed-off-by: 170210 <j170210@icloud.com>

---------

Signed-off-by: 170210 <j170210@icloud.com>
Co-authored-by: Shogo Hyodo <mmoshg8u@gmail.com>
@ulbqb

ulbqb commented Jan 5, 2024

Copy link
Copy Markdown
Contributor Author

@spoo-bar PTAL

@spoo-bar spoo-bar left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM! Thanks a lot for your contribution!

Apologies for the delay.

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.

3 participants