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

Fix attempts at cleaning content #281

Merged
merged 4 commits into from
Dec 9, 2015
Merged

Conversation

gjtorikian
Copy link
Owner

It seems that Nokogiri is overly strict and any attempt to change its strictness is not really working out.

This PR quasi reverts changes in #273, in that it simply ignores the error instead of rewriting content.

Related to github/government.github.com#423.

@gjtorikian gjtorikian changed the title Ditch attempts at cleaning content Fix attempts at cleaning content Dec 9, 2015
gjtorikian added a commit that referenced this pull request Dec 9, 2015
@gjtorikian gjtorikian merged commit a829b46 into master Dec 9, 2015
@gjtorikian gjtorikian deleted the address-nokogiri-regression branch December 9, 2015 22:16
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.

1 participant