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

Use MultiLineString::new() instead of MultiLineString() #784

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

nyurik
Copy link
Member

@nyurik nyurik commented Mar 18, 2022

Make migration simpler by using static function that will still be available if MultiLineString becomes a type alias.

Similar to #777

  • I agree to follow the project's code of conduct.
  • [ ] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

Make migration simpler by using static function that will still be available if `MultiLineString` becomes a type alias.

Similar to georust#777
@nyurik nyurik requested a review from frewsxcv March 28, 2022 02:12
nyurik added a commit to nyurik/geo that referenced this pull request Mar 28, 2022
Once georust#782 and georust#784 is merged, we should merge this PR to reflect all the new constructors and instantiation recommendations.
@nyurik nyurik mentioned this pull request Mar 28, 2022
2 tasks
@frewsxcv
Copy link
Member

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 28, 2022

Build succeeded:

@bors bors bot merged commit 1395082 into georust:main Mar 28, 2022
@nyurik nyurik deleted the from-multiline branch March 28, 2022 03:10
bors bot added a commit that referenced this pull request Mar 28, 2022
794: Add new geotype ctors to CHANGES.md r=frewsxcv a=nyurik

Once #782 and #784 is merged, we should merge this PR to reflect all the new constructors and instantiation recommendations.

- [x] I agree to follow the project's [code of conduct](https://github.com/georust/geo/blob/main/CODE_OF_CONDUCT.md).
- [x] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.
---


Co-authored-by: Yuri Astrakhan <[email protected]>
Co-authored-by: Yuri Astrakhan <[email protected]>
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