Skip to content

Conversation

@metonymic-smokey
Copy link
Member

This PR adds unit tests for the newly added disjoint relation.

Copy link
Member

@abhinavdangeti abhinavdangeti left a comment

Choose a reason for hiding this comment

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

@metonymic-smokey It appears there're some test failures here.

@abhinavdangeti abhinavdangeti added this to the v2.3.4 milestone Aug 8, 2022
github.com/bits-and-blooms/bitset v1.2.0
github.com/blevesearch/bleve_index_api v1.0.3
github.com/blevesearch/geo v0.1.13
github.com/blevesearch/geo v0.1.14-0.20220806033936-256c7049d467
Copy link
Member Author

Choose a reason for hiding this comment

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

@abhinavdangeti I am using v0.14 here so I reckon the failure isn't due to that.
Looking into it and will let you know, thanks!

Copy link
Member

Choose a reason for hiding this comment

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

Yes not related to the go.mod version. p.s. I created a v0.1.14 though.

@abhinavdangeti abhinavdangeti modified the milestones: v2.3.4, v2.3.5 Aug 18, 2022
@abhinavdangeti abhinavdangeti removed this from the v2.3.5 milestone Sep 22, 2022
@abhinavdangeti
Copy link
Member

@metonymic-smokey I see a lot of test additions here. If you want to include this for v2.3.10 - this will need rebasing and make sure the checks pass.

@metonymic-smokey
Copy link
Member Author

alright @abhinavdangeti, will let you know, thanks

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.

2 participants