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

feat: Minimal format #140

Merged
merged 3 commits into from
Dec 2, 2024
Merged

Conversation

roberth
Copy link
Contributor

@roberth roberth commented Dec 1, 2024

When including documentation into a dedicated page, we have different needs.

  • Add --anchor-prefix to avoid unnecessary/inaccurate function-library- prefix.

  • Don't render a header when description and category are empty. These can be handled by the parent document when using an "include" feature of a markdown processor / site generator.

  • Add this usage to the README. Good to help non-Nixpkgs users and let nixdoc grow outside of that context.

Also tested with #139

When including documentation into a dedicated page, we have different
needs.

- Add `--anchor-prefix` to avoid unnecessary/inaccurate `function-library-` prefix.

- Don't render a header when description and category are empty.
  These can be handled by the parent document when using an "include" feature of
  a markdown processor / site generator.

- Add this usage to the README. Good to help non-Nixpkgs users and let nixdoc
  grow outside of that context.
Copy link
Collaborator

@infinisil infinisil left a comment

Choose a reason for hiding this comment

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

Other than the need for a changelog entry, this looks good to me!

@infinisil infinisil merged commit 5a469fe into nix-community:master Dec 2, 2024
2 checks passed
@GrahamDennis
Copy link

@infinisil Is it possible to cut a release with this fix so nixdoc can be updated in nixpkgs?

@infinisil infinisil mentioned this pull request Feb 11, 2025
@infinisil
Copy link
Collaborator

@GrahamDennis Yup, here you go 🙂

@GrahamDennis
Copy link

Thanks!

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.

3 participants