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

Miscellaneous documentation refinements and additions. #307

Merged
merged 10 commits into from
Sep 7, 2022

Conversation

metasim
Copy link
Contributor

@metasim metasim commented Sep 4, 2022

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

Documentation refinements. Trimming down root content and starting to push specifics down into modules.

@metasim metasim changed the title Feature/docs refine 1 Miscellaneous documentation refinements and additions. Sep 4, 2022
@metasim metasim force-pushed the feature/docs-refine-1 branch from 73a4866 to 4626c2b Compare September 4, 2022 18:03
@metasim metasim marked this pull request as draft September 4, 2022 18:23
@metasim metasim force-pushed the feature/docs-refine-1 branch from 71c7e67 to e018bad Compare September 4, 2022 19:57
@ChristianBeilschmidt
Copy link
Contributor

Thank you for all the documentation.

In the PR that was merged, you wrote "Rust implementations of GDAL Programs" at
https://github.com/metasim/gdal/blob/7f382ea24b7efffab0f41c7a99612c227560d597/src/programs/mod.rs#L1.

But we just wrote wrappers and did not implement those programs. Maybe we should adapt this.

@metasim metasim marked this pull request as ready for review September 5, 2022 14:09
* master:
  cargo fmt
  Enabled docs.rs feature gate flag rendering.

# Conflicts:
#	src/lib.rs
src/driver.rs Show resolved Hide resolved
src/driver.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/programs/mod.rs Outdated Show resolved Hide resolved
@lnicola
Copy link
Member

lnicola commented Sep 7, 2022

bors d+

@bors
Copy link
Contributor

bors bot commented Sep 7, 2022

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

@metasim metasim force-pushed the feature/docs-refine-1 branch from 5dace40 to 840a6b6 Compare September 7, 2022 13:05
@metasim
Copy link
Contributor Author

metasim commented Sep 7, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 7, 2022

Build succeeded:

@bors bors bot merged commit a3d1d22 into georust:master Sep 7, 2022
@metasim metasim deleted the feature/docs-refine-1 branch September 7, 2022 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants