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

722 reuse code for parsing integer slices from string #747

Conversation

iButcat
Copy link
Contributor

@iButcat iButcat commented Jan 10, 2022

Closes: #722

Description


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

osmotestutils/cli_helpers.go Outdated Show resolved Hide resolved
osmotestutils/cli_helpers.go Outdated Show resolved Hide resolved
osmotestutils/cli_helpers.go Outdated Show resolved Hide resolved
Copy link
Member

@p0mvn p0mvn left a comment

Choose a reason for hiding this comment

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

Quickly reviewed it since I spent some time working on the same issue, and just have some style suggestions. Lgtm otherwise and good job

@iButcat
Copy link
Contributor Author

iButcat commented Jan 10, 2022

Thanks @akhtariev, your comment make a lot of sense here! I'm going to change that

Copy link
Member

@ValarDragon ValarDragon left a comment

Choose a reason for hiding this comment

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

Thanks @iButcat and @akhtariev for writing and reviewing this 🙏

@codecov-commenter
Copy link

Codecov Report

Merging #747 (0d0f140) into main (bc65a83) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #747   +/-   ##
=======================================
  Coverage   18.73%   18.73%           
=======================================
  Files         172      172           
  Lines       23908    23896   -12     
=======================================
  Hits         4478     4478           
+ Misses      18655    18643   -12     
  Partials      775      775           
Impacted Files Coverage Δ
x/pool-incentives/client/cli/tx.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bc65a83...0d0f140. Read the comment docs.

@ValarDragon ValarDragon merged commit daa0b7d into osmosis-labs:main Jan 12, 2022
ValarDragon pushed a commit that referenced this pull request Jan 13, 2022
* remove repeted code to create a function from it

* create function Parse Uint64 Slice From String

* replace repeted code with function made for it

* changelog add modification

* resolve style requested
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Reuse code for parsing integer slices from string
4 participants