esp-rs / esp-hal

no_std Hardware Abstraction Layers for ESP32 microcontrollers
https://docs.esp-rs.org/esp-hal/
Apache License 2.0
718 stars 194 forks source link

Documentation improvements #1221

Open jessebraham opened 7 months ago

jessebraham commented 7 months ago

There are a number of changes we would like to make to our documentation, so this will serve as a tracking issue (or an "epic", if you will) for each individual task.

If you would like to complete one of these tasks, please assign yourself to the issue if one already exists, or create a new issue and assign yourself to it.

AnthonyGrondin commented 7 months ago

Not sure where to drop this, but if we want to test for documentation coverage of the public APIs in CI, we could do something like:

cargo rustdoc -q -- -Z unstable-options --show-coverage | grep -i total | grep -oE '[0-9]*\.[0-9]*%' | head -n1 | grep -q 100.0%
  1. Get lists of documentation coverage
  2. Get the last line, "total"
  3. Extract both percentage (doc, examples) from the line.
  4. Get first one (doc)
  5. Check if == 100%.
bjoernQ commented 7 months ago

We probably also want to double check what we want to hide from the docs

e.g. here: https://docs.rs/esp32s3-hal/0.15.0/esp32s3_hal/gpio/index.html - GpioXSignals is pretty much useless for users of the HAL

https://docs.rs/esp32s3-hal/0.15.0/esp32s3_hal/gpio/trait.GpioProperties.html - the whole trait is probably cryptic and useless for most users

Not a priority but always I look into the docs I think about this and quickly forget about then

playfulFence commented 6 months ago

Working on #1276