Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Update to HtmlUnit 2.50#37

Merged
asbjornu merged 8 commits intoHtmlUnit:mainHtmlUnit/NHtmlUnit:mainfrom
AlexanderStromer:mainCopy head branch name to clipboard
Jun 4, 2021
Merged

Update to HtmlUnit 2.50#37
asbjornu merged 8 commits intoHtmlUnit:mainHtmlUnit/NHtmlUnit:mainfrom
AlexanderStromer:mainCopy head branch name to clipboard

Conversation

@AlexanderStromer
Copy link
Contributor

As requested with #34 I took it into my own hands and updated. It works fine on my side. If also ok for you, consider updating NuGet as well, thx

Had to change the tests, since wikipedia has some SSL handshake issue when requesting the page (also did not work with v2.19)

@asbjornu
Copy link
Member

asbjornu commented Jun 3, 2021

Awesome! 👏🏼 Could you please add a GitHub Action workflow as well, so we get some automated tests that prove everything is in working order? 🙏🏼

@AlexanderStromer
Copy link
Contributor Author

done, also did some badge-work for the readme
let me know your thoughts

Copy link
Member

@asbjornu asbjornu left a comment

Choose a reason for hiding this comment

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

Great stuff!

.github/workflows/unittests.yml Outdated Show resolved Hide resolved
Co-authored-by: Asbjørn Ulsberg <asbjorn@ulsberg.no>
@asbjornu asbjornu merged commit fdb3da6 into HtmlUnit:main Jun 4, 2021
@asbjornu
Copy link
Member

asbjornu commented Jun 4, 2021

Thank you so much for this, @AlexanderStromer! 🙏🏼

This was referenced Jun 4, 2021
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.