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

lib: Take advantage of section descriptions #268619

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

infinisil
Copy link
Member

@infinisil infinisil commented Nov 20, 2023

Description of changes

See https://github.com/nix-community/nixdoc/releases/tag/v2.6.0

This work is sponsored by Antithesis

Things done

  • Ensured that the descriptions render correctly in the manual.

@github-actions github-actions bot added 6.topic: module system About "NixOS" module system internals 6.topic: lib The Nixpkgs function library labels Nov 20, 2023
Comment on lines +1 to +10
/*
A partial and basic implementation of GVariant formatted strings.
See [GVariant Format Strings](https://docs.gtk.org/glib/gvariant-format-strings.html) for details.

:::{.warning}
This API is not considered fully stable and it might therefore
change in backwards incompatible ways without prior notice.
:::
*/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@linsui This now gets rendered in the Nixpkgs manual. Note that I changed the "Note" to a "Warning" here, does that sound good?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good to me. :)

@infinisil infinisil merged commit ed5e926 into NixOS:master Nov 20, 2023
22 checks passed
@infinisil infinisil deleted the lib-descriptions branch November 20, 2023 16:26
{ lib }:
# Operations on attribute sets.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why doesn't nixdoc support # as a comment character? A design choice?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think this was used to be able to distinguish between doc comments and other comments. By using /* for all doc comments it's clear what's gonna get rendered.

I guess this won't be a concern anymore once RFC 145 is implemented in nixdoc though :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: lib The Nixpkgs function library 6.topic: module system About "NixOS" module system internals 10.rebuild-darwin: 0 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants