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

relax grammar? #61

Open
pwbs opened this issue Oct 23, 2015 · 5 comments
Open

relax grammar? #61

pwbs opened this issue Oct 23, 2015 · 5 comments
Assignees
Milestone

Comments

@pwbs
Copy link
Contributor

pwbs commented Oct 23, 2015

For imported feed, it seems to me that it would be relevant to have a mode that allows "slightly broken" feeds.

I made a quick patch for my own use because I sort of have to process an invalid feed.

Is anyone interested in taking this direction? (I would not advise to accept broken feeds by default, but only when explicitly asked for)

@Chris00
Copy link
Contributor

Chris00 commented Oct 23, 2015

A relaxed mode would certainly be useful. IMHO, it should not take the path of your patch though — functorizing the whole module — but rather using an optional argument to the parsing functions.

@dinosaure
Copy link
Member

Hmmmhmm, yeah It's a good way, go to possibly deeper. And, like @Chris00, I prefer an optionnal argument.

@pwbs
Copy link
Contributor Author

pwbs commented Oct 27, 2015

I wouldn't mind using an optional argument. :)

@dinosaure
Copy link
Member

@chambart explain one solution (without GADT) to relax the compute of flow and I have a PoC of this idea. All of fields of records with possible fail is registered in relax record. And we compute theses fields with relax. It's general and sounds good, so what do you think of that ?

@dinosaure dinosaure added this to the 2.0 milestone Jan 16, 2016
@dinosaure
Copy link
Member

See #66

@dinosaure dinosaure self-assigned this Apr 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants