Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upFeature/cities polygon #21
Conversation
kiote
added some commits
Feb 22, 2016
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
kiote
Feb 25, 2016
Contributor
I commented out some tests here to restore them in a final (with regions) PR.
|
I commented out some tests here to restore them in a final (with regions) PR. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
pwalsh
Feb 26, 2016
Member
I don't like the commenting tests, and I would not normally merge code with that. But I'll do it in this case in the expectation that the next PR, which finishes the task, fixes that
|
I don't like the commenting tests, and I would not normally merge code with that. But I'll do it in this case in the expectation that the next PR, which finishes the task, fixes that |
added a commit
that referenced
this pull request
Feb 26, 2016
pwalsh
merged commit 89173c3
into
openspending:master
Feb 26, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
kiote commentedFeb 25, 2016
This PR represents city polygon endpoint and some refactoring of previous (country polygon) code to make it more reusable.