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.

Sign up for GitHub

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

Jump to bottom

Review naming of high-level PyOWM interface methods #20

Closed
csparpa opened this issue May 3, 2014 · 1 comment
Closed

Review naming of high-level PyOWM interface methods #20

csparpa opened this issue May 3, 2014 · 1 comment
Assignees
@csparpa
Labels
enhancement refactoring
Milestone
Release 2.0

Comments

@csparpa
Copy link
Owner

csparpa commented May 3, 2014

Class OWM25 is under focus. Make sure the function names are clear and convey what they really do.

This task could lead to a change in the library interface: this means that users must be notified of this non retro compatibility and docs must be updated, as well as code usage wiki and example snippets, and README.md

This also includes changes introduced with #17
@csparpa csparpa added this to the Release 2.0 milestone May 3, 2014
@csparpa csparpa self-assigned this May 3, 2014
@csparpa
Copy link
Owner Author

csparpa commented Aug 26, 2014

Closed with PR #33

@csparpa csparpa closed this as completed Aug 26, 2014
csparpa added a commit that referenced this issue Aug 26, 2014
@csparpa
Merge pull request #33 from csparpa/feature/#20_naming_review
3231445 
Feature/#20 naming review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@csparpa csparpa

Labels
enhancement refactoring
Projects
None yet
Milestone
Release 2.0
Development

No branches or pull requests
1 participant
@csparpa