-
-
Notifications
You must be signed in to change notification settings - Fork 420
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
Mentions of "URI" should be "URL" #80
Comments
+1 |
This is a good idea! However, it will not be fixed in the next version of requests-oauthlib because I want 0.3.4 to make people's lives as easy as possible. =) |
Think the only place this would break would be the |
@ib-lundgren Not a bad idea: we should also raise a new issue to remind us to get rid of it eventually. =) |
@Lukasa @ib-lundgren As a newcomer, I would like to work upon this issue. Can I ? |
a) I agree that a uniform use of one of Why b)?
|
The acronym
uri
is used all over the place in code and documentation, where it should beurl
. Alsourl
is used in a bunch of places too, where it is already correct.This is a semantic nitpick, and probably not worth breaking backwards compatibility over. But worth keeping in mind in case you ever end up doing a big refactor and see an opportunity to clean this up. :)
Feel free to close this bug if it doesn't make sense to pursue at this time.
The text was updated successfully, but these errors were encountered: