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

Release 2.0.0 #103

Merged
merged 4 commits into from
May 9, 2017
Merged

Release 2.0.0 #103

merged 4 commits into from
May 9, 2017

Conversation

thiemowmde
Copy link
Contributor

Please have a look at the 2.0.0 milestone. Because we changed a hash calculation in an incompatible way, we need to do a major version bump anyway. We might as well include all previously proposed breaking changes.

@thiemowmde thiemowmde added this to the 2.0.0 milestone Apr 25, 2017
@thiemowmde thiemowmde requested a review from manicki April 25, 2017 09:47
@thiemowmde thiemowmde force-pushed the release200 branch 2 times, most recently from a30e953 to 6465af3 Compare May 4, 2017 11:46
@JeroenDeDauw
Copy link
Member

Might want to consider #108

README.md Outdated

* `GlobeCoordinateValue` does not accept empty strings as globes any more.
* `GlobeCoordinateValue` does not accept precisions outside the [-360..+360] interval any more.
* Changed hash calculation of `GlobeCoordinateValue` to be more robust.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this explicitly say the hash change was a backwards incompatible one? I know "robust" is supposed to mean it here but if does not immediately strike as a breaking change IMO.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

README.md Outdated
@@ -93,8 +93,11 @@ It is based upon and contains a lot of code written by [Jeroen De Dauw]

## Release notes

### 2.0.0 (dev)
### 2.0.0 (2017-05-08)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deprecation still missing

@manicki manicki merged commit 25bae64 into master May 9, 2017
@manicki manicki deleted the release200 branch May 9, 2017 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants