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

Validate the resulting feed #21

Merged
merged 1 commit into from May 13, 2015
Merged

Validate the resulting feed #21

merged 1 commit into from May 13, 2015

Conversation

benbalter
Copy link
Contributor

Both by parsing the RSS for expectations, and by calling the W3 validation API.

To note, there are some gems out there, but they haven't been updated in 3+ years, and given one call, things were simple enough to implement with a post request + nokigiri.

Fixes #16

@benbalter benbalter changed the title Spike out validation Validate the resulting feed May 13, 2015
@parkr
Copy link
Member

parkr commented May 13, 2015

Nifty!

@pathawks
Copy link
Member

🆒

benbalter added a commit that referenced this pull request May 13, 2015
Validate the resulting feed
@benbalter benbalter merged commit 31ef200 into master May 13, 2015
@benbalter benbalter deleted the validation branch May 13, 2015 17:41
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Validate feed
4 participants