-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Update examples to use org.htmlunit:htmlunit #42112
Comments
for context: I made comments on 3.8 product branch, I suppose my comments are valid for 3.8 community branch, but I didn't check |
@rolfedh @michalvavrik, AFAIK, the package change for HTML unit occurred in 3.12.0, #40765. |
Last release I can see was https://mvnrepository.com/artifact/net.sourceforge.htmlunit/htmlunit year and half ago. You are absolutely correct that Quarkus uses Anyway, I can see it is hardly important, I apologize for bringing it up @rolfedh , we can probably just close it as far as 3.8 is related. |
I found a couple of instances that needed updating in |
@sberyozkin @michalvavrik It's easy enough for me to make similar updates in |
Description
@michalvavrik commented that examples with the following text should be updated to use https://mvnrepository.com/artifact/org.htmlunit/htmlunit instead:
These include the following results for "net.sourceforge.htmlunit" in "*.adoc"
/home/rdlugyhe/quarkus/docs/src/main/asciidoc/security-oidc-code-flow-authentication.adoc:
1503: net.sourceforge.htmlunit
1523: testImplementation("net.sourceforge.htmlunit:htmlunit")
/home/rdlugyhe/quarkus/docs/src/main/asciidoc/security-openid-connect-multitenancy.adoc:
436: net.sourceforge.htmlunit
447: testImplementation("net.sourceforge.htmlunit:htmlunit")
fyi: @sberyozkin
Implementation ideas
No response
The text was updated successfully, but these errors were encountered: