Skip to content

Conversation

@MaverickWingman
Copy link
Contributor

@MaverickWingman MaverickWingman commented Mar 28, 2024

Pull request title

  • Changed the HtmlUnit from 2.x.x to HtmlUnit 3.x.x
  • Upgrade to Htmlunit 3.11.0 (Latest release February 10, 2024)

Pull request description

    1. adjust maven group id to org.htmlunit

      org.htmlunit
      htmlunit
      3.11.0

      Replace all usages of com.gargoylesoftware.htmlunit by org.htmlunit.

      Replace all usages of net.sourceforge.htmlunit by org.htmlunit.

      a. import org.htmlunit.BrowserVersion;
      b. import org.htmlunit.WebClient;
      c. import org.htmlunit.html.HtmlInput;
      d. import org.htmlunit.html.HtmlPage;

  • Reference: https://htmlunit.sourceforge.io/

For detailed contributing instructions see https://github.com/iluwatar/java-design-patterns/wiki/01.-How-to-contribute

@MaverickWingman
Copy link
Contributor Author

Hey @iluwatar, Help me with this thing also.

Thankyou!

Screenshot 2024-03-28 at 3 52 00 PM Screenshot 2024-03-28 at 3 52 33 PM

@iluwatar
Copy link
Owner

The code style does not follow the google style that we use.

See https://github.com/iluwatar/java-design-patterns/wiki/12.-IDE-instructions

You can run the checkstyle checks using IDE plugin or using the command line (mvn validate).

Comment on lines 12 to 28
<<<<<<< Updated upstream
=======
allow:
- dependency-name: "org.springframework.boot:spring-boot-dependencies"
- dependency-name: "commons-dbcp:commons-dbcp"
- dependency-name: "org.htmlunit.:htmlunit"
- dependency-name: "com.google.code.gson:gson"
- dependency-name: "com.google.inject:guice"
- dependency-name: "com.github.stefanbirkner:system-lambda"
- dependency-name: "org.apache.maven.plugins:maven-surefire-plugin"
- dependency-name: "org.apache.maven.plugins:maven-assembly-plugin"
- dependency-name: "org.sonarsource.scanner.maven:sonar-maven-plugin"
- dependency-name: "org.apache.maven.plugins:maven-checkstyle-plugin"
- dependency-name: "com.mycila:license-maven-plugin"
- dependency-name: "org.jacoco:jacoco-maven-plugin"
- dependency-name: "com.iluwatar.urm:urm-maven-plugin"
>>>>>>> Stashed changes
Copy link
Owner

Choose a reason for hiding this comment

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

This should be removed

@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@iluwatar
Copy link
Owner

Looks good! Thank you for the contribution 🎉

@all-contributors please add @MaverickWingman for code

@allcontributors
Copy link
Contributor

@iluwatar

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

I've put up a pull request to add @MaverickWingman! 🎉

@iluwatar iluwatar merged commit baa7793 into iluwatar:master Mar 29, 2024
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.

2 participants