Skip to content

Fixes #656 usage strings to be in sync with command parsing#660

Merged
snazha-blkio merged 2 commits intoLFDT-web3j:release/4.0from
taivokasper:fix-usage
Oct 26, 2018
Merged

Fixes #656 usage strings to be in sync with command parsing#660
snazha-blkio merged 2 commits intoLFDT-web3j:release/4.0from
taivokasper:fix-usage

Conversation

@taivokasper
Copy link
Copy Markdown
Contributor

Usage string and actual usage were not in sync. I think changing the usage to reflect actual usage is the only way to go for it to stay backwards compatible.

@taivokasper
Copy link
Copy Markdown
Contributor Author

taivokasper commented Aug 4, 2018

Is the travis fail a fluke? Running the same failing command locally passes.

I'm gonna try retriggering the checks.

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 5, 2018

Codecov Report

Merging #660 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #660   +/-   ##
=========================================
  Coverage     77.13%   77.13%           
  Complexity     1793     1793           
=========================================
  Files           239      239           
  Lines          6664     6664           
  Branches        996      996           
=========================================
  Hits           5140     5140           
  Misses         1272     1272           
  Partials        252      252
Impacted Files Coverage Δ Complexity Δ
...j/codegen/TruffleJsonFunctionWrapperGenerator.java 39.04% <ø> (ø) 7 <0> (ø) ⬇️
...eb3j/codegen/SolidityFunctionWrapperGenerator.java 59.61% <ø> (ø) 4 <0> (ø) ⬇️

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 8349907...f1e494b. Read the comment docs.

@snazha-blkio snazha-blkio changed the base branch from master to release/4.0 October 18, 2018 20:40
@snazha-blkio snazha-blkio added this to the 4.0 milestone Oct 18, 2018
Copy link
Copy Markdown
Contributor

@conor10 conor10 left a comment

Choose a reason for hiding this comment

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

Thanks @taivokasper!

@snazha-blkio snazha-blkio merged commit 571a5ec into LFDT-web3j:release/4.0 Oct 26, 2018
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