geoarrow-rs icon indicating copy to clipboard operation
geoarrow-rs copied to clipboard

Deny missing documentation and flesh out existing docs

Open gadomski opened this issue 1 year ago • 2 comments

By adding #![deny(missing_docs)] to the Rust crate, we can help ensure that all public items are documented. We should also add examples, when appropriate, to existing docs.

Checklist

  • [x] Document lib: https://github.com/geoarrow/geoarrow-rs/pull/696
  • [x] Document "shallow" mods: trait_, error, datatypes, table, chuncked_array, indexed: https://github.com/geoarrow/geoarrow-rs/pull/696
  • [x] Add #![deny(missing_docs)] to top level and #![allow(missing_docs)] (or warn) to "deep" mods: https://github.com/geoarrow/geoarrow-rs/pull/696
  • [ ] Document scalar
  • [ ] Document io
  • [ ] Document geo_traits
  • [ ] Document array
  • [ ] Document algorithm

gadomski avatar Aug 11 '24 12:08 gadomski

Is it possible to add this for sub-crates to start? E.g. not at the top-level lib.rs but inside specific modules, and then we can slowly work our way up to documenting everything?

kylebarron avatar Aug 11 '24 14:08 kylebarron

Yup, it is, or you can #![allow(missing_docs)] on a problematic module too.

gadomski avatar Aug 11 '24 14:08 gadomski

All current stable crates deny missing documentation.

kylebarron avatar Aug 05 '25 19:08 kylebarron