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.

Sign up for GitHub

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

Jump to bottom

fix some doc issues #678

Merged

pjfanning merged 3 commits into apache:main from pjfanning:http-license

Jun 3, 2024

Contributor

pjfanning commented Jun 1, 2024

doesn't fix everything in #677 but is a start

use Java 8 to build like we do when we publish the docs - the Java API links work when Java 8 is used but the incorrect paths are used when Java 11 is used to build the docs
we have very little control over all the mistakes in the generated license report

pjfanning added 3 commits

June 1, 2024 20:09


          http links in license report

87c284a


          try to fix some of the link issues in the docs

0cd5493


          Update build.sbt

ae545b5

pjfanning requested review from Roiocam, raboof, mdedetrich, nvollmar, samueleresca and He-Pin

June 3, 2024 13:04

nvollmar approved these changes

View reviewed changes

Contributor

nvollmar left a comment

lgtm

pjfanning merged commit fd20f64 into apache:main

51 checks passed

pjfanning deleted the http-license branch

June 3, 2024 13:20

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

Reviewers

nvollmar nvollmar approved these changes

Roiocam Awaiting requested review from Roiocam

raboof Awaiting requested review from raboof

mdedetrich Awaiting requested review from mdedetrich

samueleresca Awaiting requested review from samueleresca

He-Pin Awaiting requested review from He-Pin

Labels

None yet