-
Notifications
You must be signed in to change notification settings - Fork 40
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
remove external module dependencies #55
Conversation
addon.xml
Outdated
@@ -1,15 +1,11 @@ | |||
<?xml version="1.0" encoding="UTF-8" standalone="yes"?> | |||
<addon id="metadata.themoviedb.org.python" | |||
name="The Movie Database Python" | |||
version="1.2.1" | |||
version="1.2.2" |
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.
I don't think this is a bugfix version :D
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.
Fair enough. I've always thought of the versioning as major.minor.maintenance and this feels like maintenance. But if 1.3.0 would be better, that's fine with me.
changelog.txt
Outdated
@@ -1,3 +1,8 @@ | |||
v1.2.2 (2020-10-04) | |||
- Fix: removed dependencies on requests, tmdbsimple, and trakt modules |
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.
I don't think these should be Fix
?
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.
I was trying to keep the changelog to the same format, and there were only Features and Fixes. Features felt like user facing things, so I used Fixes. Could we just list it as Change?
Yes, I think that's sensible
…On Tue, Oct 6, 2020, 00:47 Kyle Johnson ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In changelog.txt
<#55 (comment)>
:
> @@ -1,3 +1,8 @@
+v1.2.2 (2020-10-04)
+- Fix: removed dependencies on requests, tmdbsimple, and trakt modules
I was trying to keep the changelog to the same format, and there were only
Features and Fixes. Features felt like user facing things, so I used Fixes.
Could we just list it as Change?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#55 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABNLEZC7OAHPCQ4WA63FH6DSJJEH3ANCNFSM4SD5CBWQ>
.
|
3a42bdf updates the change log and version number. I also added the change log info to the news section of addon.xml |
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.
…ion-providers-metadata-tvshows-themoviedb-org-python
This is the bundle of code changes to remove the dependencies on tmdbsimple, trakt, and requests modules. All the external calls are now handled by urllib2 (Python 2) or urllib (Python 3). I tried to minimize the changes to the core of the code, so tmdbapi.py mostly mimics the previous API calls and the changes to traktratings.py, imdbratings.py, and fanarttv.py are all internal. There are two sets of changes to tmdb.py.