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.

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

Dev1.2.1 #76

Merged
merged 10 commits into from
Feb 26, 2015
Merged

Dev1.2.1 #76

merged 10 commits into from
Feb 26, 2015

Conversation

stolstov
Copy link
Member

@stolstov stolstov commented Feb 6, 2015

@climbage
Copy link
Member

@stolstov This looks good to me.

@randallwhitman
Copy link
Contributor

Go ahead - my tests pass.
If you want, one detail in a comment in NonSimpleResult: s/interioir/interior/ .

@stolstov
Copy link
Member Author

@randallwhitman What detail?

@randallwhitman
Copy link
Contributor

An extra 'i' in misspelled "interior" in the comment for OGCPolygonSelfTangency, line 80.

@stolstov
Copy link
Member Author

@randallwhitman I see... We'll fix this next time.

@randallwhitman
Copy link
Contributor

Confirm that the ST_Geometry/Hive test scripts run OK after the recent commit.

@alocke
Copy link
Member

alocke commented Feb 23, 2015

All tests pass.

@stolstov
Copy link
Member Author

@climbage Lets merge?

@climbage
Copy link
Member

@stolstov sounds good. I can push to maven after.

stolstov added a commit that referenced this pull request Feb 26, 2015
@stolstov stolstov merged commit a942691 into master Feb 26, 2015
@stolstov stolstov deleted the dev1.2.1 branch February 26, 2015 18:31
@stolstov
Copy link
Member Author

@alocke I've assigned bunch of issues to verify to you if you do not mind:)

@climbage
Copy link
Member

Sure. I'm in no hurry.

@alocke
Copy link
Member

alocke commented Mar 12, 2015

@climbage I have verified all the fixes that were merged into master.

@climbage
Copy link
Member

Sounds good. I'll push to maven.

@stolstov
Copy link
Member Author

@climbage Here is url for fixed issues to add to the release description: https://github.com/Esri/geometry-api-java/issues?q=is%3Aissue+is%3Aclosed+milestone%3A1.2.1

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.

None yet

4 participants