Skip to content
This repository has been archived by the owner on Apr 2, 2022. It is now read-only.

Bump to 2021, WKT to 0.10, linting #10

Merged
merged 2 commits into from
Mar 19, 2022
Merged

Bump to 2021, WKT to 0.10, linting #10

merged 2 commits into from
Mar 19, 2022

Conversation

nyurik
Copy link
Member

@nyurik nyurik commented Mar 14, 2022

  • Bump Rust edition to 2021
  • Address a few cargo clippy suggestions
  • Bump WKT dependency 0.9 -> 0.10
  • few minor spelling fixes
  • 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.

* Bump Rust edition to 2021
* Address a few `cargo clippy` suggestions
* Bump WKT dependency 0.9 -> 0.10
* few minor spelling fixes
@nyurik nyurik requested review from michaelkirk and rmanoka March 14, 2022 16:03
Copy link
Contributor

@rmanoka rmanoka left a comment

Choose a reason for hiding this comment

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

lgtm!

@michaelkirk
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Mar 15, 2022
10: Bump to 2021, WKT to 0.10, linting r=michaelkirk a=nyurik

* Bump Rust edition to 2021
* Address a few `cargo clippy` suggestions
* Bump WKT dependency 0.9 -> 0.10
* few minor spelling fixes

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



Co-authored-by: Yuri Astrakhan <[email protected]>
@bors
Copy link
Contributor

bors bot commented Mar 15, 2022

Build failed:

@nyurik
Copy link
Member Author

nyurik commented Mar 15, 2022

oops, need to bump the minimal required version, sec

@nyurik
Copy link
Member Author

nyurik commented Mar 15, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 15, 2022

🔒 Permission denied

Existing reviewers: click here to make nyurik a reviewer

@nyurik
Copy link
Member Author

nyurik commented Mar 15, 2022

i'm not worthy, will delegate to others to +r it

@nyurik nyurik requested a review from frewsxcv March 16, 2022 19:56
@nyurik
Copy link
Member Author

nyurik commented Mar 16, 2022

I bumped the min rust ver to match the geo project, please re-bors it

@rmanoka
Copy link
Contributor

rmanoka commented Mar 19, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 19, 2022

Build succeeded:

@bors bors bot merged commit b584dae into georust:master Mar 19, 2022
@nyurik nyurik deleted the bump branch March 19, 2022 19:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants