-
Notifications
You must be signed in to change notification settings - Fork 49
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
Cleanup following #35 #38
Conversation
src/channel.rs
Outdated
/// | ||
/// Channel::from_url(url).unwrap(); | ||
/// let channel = input.parse::<Channel>().unwrap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change should be reverted since this doc is describing the from_url
method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, I'll fix that one up.
Minus my comment, this all looks good to me! Regarding the @PhnxRbrn Have any thoughts about these changes? |
@frewsxcv Any rss reader would use the from_url method, so to me it does not make much sense behind a feature gate |
@PhnxRbrn My thought is that an rss application may want to use their own networking library or customize the request used to retrieve the rss feed. If a user doesn't want to use this feature then we're including a pretty large dependency ( |
@jameshurst very true, with that in mind, it does make sense to keep it behind the flag. |
thanks again everyone, going to merge and publish this :) |
No description provided.