Skip to content

Add Geometry@toSqlExpression method #70

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

Merged
merged 8 commits into from
Dec 12, 2022
Merged

Conversation

Synchro
Copy link
Contributor

@Synchro Synchro commented Dec 7, 2022

Here's a PR adding a method (with a test!) that generates an Expression that can be used for manual casting along the lines of what I was asking in #69. I'm not sure about the name, or whether you think this is incomplete!

Copy link
Owner

@MatanYadaev MatanYadaev left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. I left a few comments.

@Synchro
Copy link
Contributor Author

Synchro commented Dec 12, 2022

I've done all those (and it's much cleaner) - but I don't see why this merge conflict is happening when it's a straight addition

@MatanYadaev
Copy link
Owner

@Synchro I updated the GeometryTest on master, please rebase and fix the conflicts. It's should be simple.

@MatanYadaev MatanYadaev changed the title Add a manual casting method, fixes #69 Add Geometry@toSqlExpression method Dec 12, 2022
@MatanYadaev MatanYadaev merged commit bfb9a4f into MatanYadaev:master Dec 12, 2022
@MatanYadaev
Copy link
Owner

@Synchro Fixed the conflicts. 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