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

Add set description #212

Merged
merged 3 commits into from
Sep 1, 2021
Merged

Add set description #212

merged 3 commits into from
Sep 1, 2021

Conversation

msalib
Copy link
Contributor

@msalib msalib commented Sep 1, 2021

  • 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.

This is a really simple change that allows users to set descriptions on major GDAL objects; that's mostly useful for RasterBands. If you prefer I could just make this a method on RasterBand only instead of more generally making it part of the Metadata trait.

@lnicola
Copy link
Member

lnicola commented Sep 1, 2021

Thanks! And don't sigh.

bors r+

bors bot added a commit that referenced this pull request Sep 1, 2021
212: Add set description r=lnicola a=msalib

- [X] I agree to follow the project's [code of conduct](https://github.com/georust/gdal/blob/master/CODE_OF_CONDUCT.md).
- [X] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.
---
This is a really simple change that allows users to set descriptions on major GDAL objects; that's mostly useful for RasterBands. If you prefer I could just make this a method on `RasterBand` only instead of more generally making it part of the `Metadata` trait.


Co-authored-by: Michael Salib <[email protected]>
@lnicola
Copy link
Member

lnicola commented Sep 1, 2021

bors r-

Well, while we're at sighing, can you add a link to the PR in the changelog?

bors d+

@bors
Copy link
Contributor

bors bot commented Sep 1, 2021

✌️ msalib can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@bors
Copy link
Contributor

bors bot commented Sep 1, 2021

Canceled.

@msalib
Copy link
Contributor Author

msalib commented Sep 1, 2021

Sure thing!

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 1, 2021

@bors bors bot merged commit 2024aee into georust:master Sep 1, 2021
@msalib msalib deleted the add-set_description branch September 1, 2021 17:48
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