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

Improve acap-logging docs #41

Merged
merged 4 commits into from
Jul 10, 2024
Merged

Improve acap-logging docs #41

merged 4 commits into from
Jul 10, 2024

Conversation

apljungquist
Copy link
Contributor

The crate level documentation is taken from README.md to make it easier to ensure that crates.io, docs.rs and GitHub all show sensible documentation with minimal effort.

Also update the hello_world example since it would not display any output by default when run in a terminal. To increase the utility of the app a bit further without increasing the complexity it is used to also demonstrate the effect of logging at other levels and of printing.

The crate level documentation is taken from `README.md` to make it
easier to ensure that crates.io, docs.rs and GitHub all show sensible
documentation with minimal effort.

Also update the `hello_world` example since it would not display any
output by default when run in a terminal. To increase the utility of
the app a bit further without increasing the complexity it is used to
also demonstrate the effect of logging at other levels and of printing.
@apljungquist apljungquist requested a review from a team as a code owner July 10, 2024 16:23
Also add a table of library crates provided by this project and where
to find their documentation. This makes it easier to get an overview of
what is available and where.
The documentation test is annotated with the `no_run` attribute because
it fails to create a unix socket logger in GitHub Actions. In GitHub
this causes it to not be syntax highlighted but I'm hoping crates.io
will pick it up anyway.
@apljungquist apljungquist merged commit 302367e into main Jul 10, 2024
1 check passed
@apljungquist apljungquist deleted the better_acap_logging_docs branch July 10, 2024 17:57
apljungquist added a commit that referenced this pull request Jul 19, 2024
* upstream/main:
  Generate app `LICENSE` files (#44)
  Add acap-ssh-utils bin and lib crate (#19)
  Add cargo-acap-build tool (#25)
  Remove partially containerized workflow (#42)
  Improve acap-logging docs (#41)
  chore: Prepare logging library crate for publishing (#40)
  feat: Add metadata broker wrapper and example (#22)
  Rename environment variables controlling the Makefile (#26)
  Adopt dev container as primary environment (#39)
  Make the `build` verb build for only one target (#31)
  chore(deps): bump ghcr.io/devcontainers/features/common-utils (#38)
  chore(deps): bump log from 0.4.21 to 0.4.22 in the default group (#37)
  Add dev container (#36)
  Add link to acap-rs-app-template in `README.md` (#34)
  Document `python3-venv` dependency (#35)
  fix: Repair `Makefile` (#32)
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.

1 participant