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

fix(doc): link to CommandLine.java #553

Merged
merged 1 commit into from
Nov 21, 2018
Merged

fix(doc): link to CommandLine.java #553

merged 1 commit into from
Nov 21, 2018

Conversation

simon04
Copy link
Contributor

@simon04 simon04 commented Nov 21, 2018

When viewing https://picocli.info/, the link resolves to the non-existing https://picocli.info/src/main/java/picocli/CommandLine.java

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #553   +/-   ##
=========================================
  Coverage     89.17%   89.17%           
  Complexity      285      285           
=========================================
  Files             4        4           
  Lines          3946     3946           
  Branches        967      967           
=========================================
  Hits           3519     3519           
  Misses          213      213           
  Partials        214      214

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 03412d3...b04107b. Read the comment docs.

@remkop remkop merged commit 9d086d9 into remkop:master Nov 21, 2018
@remkop
Copy link
Owner

remkop commented Nov 21, 2018

Thank you for the pull request!
I merged it into master. Will update the link in the generated HTML when I get home.

@remkop remkop added this to the 3.8.1 milestone Nov 21, 2018
remkop added a commit that referenced this pull request Nov 28, 2018
@remkop
Copy link
Owner

remkop commented Nov 28, 2018

Finally updated the generated HTML as well... Sorry for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants