search

Cumulus / Syndic

RSS and Atom feed parsing
MIT License
34 stars 13 forks source link

Error with ocaml.org #54

Closed dinosaure closed 9 years ago

dinosaure commented 9 years ago

I try to create a tests suites for CI but Syndic raise an error with http://ocaml.org/feed.xml (the only test for the moment). The bug is: document MUST contains exactly one <channel> element. Something is wrong in Syndic.Atom. I'll search the bug. We can see the tests suites in test branch.

PS: I have a new monitor for my computer !

Chris00 commented 9 years ago

On Sun, 5 Apr 2015 09:18:19 -0700, Calascibetta Romain wrote:

I try to create a tests suites for CI but Syndic raise an error with http:// ocaml.org/feed.xml (the only test for the moment). The bug is: document MUST contains exactly one element. Something is wrong in Syndic.Atom. I'll search the bug.

Looks like you are tying to parse the feed with RSS2. This test is a good idea, I get:

Syndic_error.Error ((279, 1661), "Category elements MUST have a 'term' attribute").

This indeed needs to be investigated.

PS: I have a new monitor for my computer !

Glad for you!

Chris00 commented 9 years ago

@dinosaure I am inclined to release version 1.3.1 for the feed of the website to be correct but you may want to finish your test suite first. Let me now.

dinosaure commented 9 years ago

Yes, the test suite it's not mandotary for the next release - it's just for us. So, you can release 1.3.1 now :+1: !