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

Prepare for release 0.0.4 #47

Merged
merged 20 commits into from
Jan 15, 2022
Merged

Prepare for release 0.0.4 #47

merged 20 commits into from
Jan 15, 2022

Conversation

lukas-h
Copy link
Member

@lukas-h lukas-h commented Dec 14, 2021

No description provided.

lukas-h and others added 17 commits November 22, 2021 23:00
This commit also renames Geometry to GeometryObject
This allows PRs from forks to still run the tests successfully
The coverage job is allowed to continue on error, letting the entire
workflow 'pass' if we hit issues running or reporting coverage
This should resolve an issue where the 'comment code coverage on PR'
action fails after being merged into main. See
romeovs/lcov-reporter-action#16
…rovements (#43)

* #19, #20, #21, refactor

* convert GeoJSONObjectType to enum, rename

* rm overridden bboxes

* Implement vector operations #23

* statically type 'round' helper func, fix unit test

* gen coverage

* fix typo

* geojson unit tests, type fixes

* fix deserialization with GeoJSONObjectType, implement hashCode

* initial unit tests for new constructors

* integration tests

* fix path

* test bbox on other geojson types

* update enum tests

* required params for new constructors, unit tests

* license notices
@lukas-h lukas-h marked this pull request as ready for review December 19, 2021 15:26
@lukas-h
Copy link
Member Author

lukas-h commented Jan 15, 2022

I think it's urgent to release a new version, because in 0.0.3 we have some serialization errors that are now fixed.

The merging (main -> releases) I had to do was a little bit messy this time..

Copy link
Collaborator

@tobrun tobrun left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@lukas-h lukas-h merged commit fca43cf into releases Jan 15, 2022
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.

3 participants