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

Feature/139 html node.write virtual #140

Merged

Conversation

DzonnyDZ
Copy link

@DzonnyDZ DzonnyDZ commented Feb 2, 2018

This is solving issue #139

Hello there,
We have been using HtmlAgilityPack in our company for several years. Thinking it's dead we maintained our own copy. Now I see you started to take care of it and I would like the changes we have done to be integrated to the main repository. This is the smallest one.
We did several changes (or rather fixes) to HtmlAgilityPack code especially to fix compatibility issues with XSLT and XPath. Also to solve sometimes some issues with encoding. So unless these issues are already fixed, I would make more pull requests - if this is the way we can cooperate. If you have any other suggestion, I'm in.

@JonathanMagnan JonathanMagnan self-assigned this Feb 4, 2018
@JonathanMagnan
Copy link
Member

Hello @DzonnyDZ ,

Thank you for the pull, we will make the merge this week.

Feel free to provide more pull but please keep them short. While we are happy to support this library, we can only put a few hours every week on it.

Do you think we could have access to your main HtmlAgilityPack repository? Perhaps we could look on all changes your have done so far.

Best Regards,

Jonathan

@DzonnyDZ
Copy link
Author

DzonnyDZ commented Feb 5, 2018

Hello,
Unfortunately I cannot provide you access to the repository where our changes are. But I uploaded the changes here: https://github.com/DzonnyDZ/HtmlAgilityPack-RFE. But be aware that these are the changes done to some old version of HTML Agility Pack and I don't think they can be just merged with current status of the code.

My idea was that I will now do tests and see if the latest version of HtmlAgilityPack contains any bugs which we have been fixing. And in case of bugs found I'd make additional pull requests. Unfortunately I also cannot fully dedicate to this, so I cannot speak how fast I will do that. I'll definitely keep the pull requests short.

I think HtmlAgilityPack is great library and I definitely want to use it in future, so I want it to support all possible scenarios and I want to contribute to that.
Thank you for taking care of HAP.

@JonathanMagnan
Copy link
Member

Hello @DzonnyDZ ,

Thank you, I will have my programmer on Thursday to compare both repositories and check if there is some easy fix/merge he can do.

I guess we will have a better idea when he will start to check your repository.

No problem, we try our best to add value to the .NET Community ;)

Best Regards,

Jonathan

@JonathanMagnan JonathanMagnan merged commit 5e61a0f into zzzprojects:master Feb 12, 2018
@JonathanMagnan
Copy link
Member

Hello @DzonnyDZ ,

The v1.6.17 has been released.

We also reviewed your repository and some fix will be probably added soon from it ;)

Thank a lot.

Best Regards,

Jonathan

@DzonnyDZ DzonnyDZ deleted the Feature/139-HtmlNode.Write-virtual branch February 13, 2018 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants