Skip to content

Request to pull Rotten Tomatoes plugin changes#1

Merged
paranoidi merged 6 commits intoFlexget:masterfrom
jawilson:master
Feb 17, 2012
Merged

Request to pull Rotten Tomatoes plugin changes#1
paranoidi merged 6 commits intoFlexget:masterfrom
jawilson:master

Conversation

@jawilson
Copy link
Copy Markdown
Member

@jawilson jawilson commented Feb 1, 2012

I've written a Rotten Tomatoes plugin based on the existing TMDB and IMDB plugins. It works pretty well. Let me know if there's anything I need to do.

Thanks!

@jawilson
Copy link
Copy Markdown
Member Author

jawilson commented Feb 2, 2012

I found a bug where it will always perform an RT lookup, even if it has the imdb_id. I'll fix up a workaround but I may not get to it until this weekend.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

module level constants should be in upper case, although looks like we don't have that everywhere anyway ... :)

@paranoidi
Copy link
Copy Markdown
Member

Looks very good, only thing that would make this superb is (minimal) unit tests.

@jawilson
Copy link
Copy Markdown
Member Author

So are you just waiting on RST comments and some unit tests to accept?

paranoidi added a commit that referenced this pull request Feb 17, 2012
Request to pull Rotten Tomatoes plugin.
@paranoidi paranoidi merged commit 5625fe9 into Flexget:master Feb 17, 2012
gazpachoking pushed a commit that referenced this pull request Oct 29, 2014
gazpachoking pushed a commit that referenced this pull request Jun 13, 2015
Rework wait for magenetization based on input from Gazpachoking.
gazpachoking pushed a commit that referenced this pull request Oct 22, 2015
cvium added a commit that referenced this pull request Nov 10, 2015
stevezau added a commit that referenced this pull request Nov 11, 2017
@metaMMA metaMMA mentioned this pull request Nov 7, 2019
5 tasks
gazpachoking pushed a commit that referenced this pull request Feb 16, 2021
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.

3 participants