-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implement mdBook-Based Documentation for autofirma-nix #43
Comments
A first stab at this is in the branch main, under /cc: @panchoh & @CesarGallego for review |
Nice!!! At first glance, I’d suggest using sentence case instead of title case, as it is used (almost fully consistenly) in the stylix guide. Home-Manager Moduleto be rendered as: Home Manager module(Note that Home Manager as a project is spelled without the hyphen). With title case, it would be rendered as: Home Manager Modulewhich seems less clear to me since it detracts from the fact that Home Manager is a proper noun. I’ve also noticed that in the options, the default values that have the with applied configuration litany, are backtick-quoted, possibly to distinguish the values from the litany, since they are rendered in the same font, but I wonder if it would be possible/desirable to render them like this:
Finally, I noticed that on the troubleshooting section, on one line the tilde is used to refer to the home directory, and on the other, I’ve yet to read the whole documentation thoroughly, but I thought that sending these your way early would be beneficial. Awesome job, @nilp0inter ! |
Done!
Nothing we can do about that, unfortunately.
Noted. I am not changing that already because I suspect we'll be changing the whole section. The ChatGPT rewrite took a higher than expected toll on the content. :(
Me too 👯 Thank you for the review! |
I've reviewed the rest of the doc, and looks to me it’s ready for prime time! |
I think some examples are not working. We should test them before publication. Did you, by any chance, try them? |
Ah, no, I didn't. |
Reviewing that README was injected to the mdDoc sources. Save for the following, all info has been preserved (not taking into account the possibly broken code examples): The information on these paragraphs has gone into oblivion: The integration of AutoFirma in Home Manager enables the Additionally, you can enable DNIe integration, including NFC-based DNIe from an @nilp0inter, how do you want to proceed? |
Thanks, @panchoh! Please don't hesitate to commit the missing bits directly where you see fit. |
Summary
The
autofirma-nix
project has grown in scope and complexity, making our current README quite cluttered. To improve the user experience and maintainability, we should create a dedicated documentation site using mdBook—similar to what stylix does. This new documentation will be published via GitHub Pages, with an automated workflow to keep it up to date.Why This Is Needed
Proposed Approach
Set Up mdBook
docs
folder (orbook
folder, naming is flexible) with anmdBook
configuration.SUMMARY.md
to outline the structure (e.g., Installation, Configuration, Known Issues, FAQ).Draft the Content
md
files.Automate Publishing via GitHub Pages
gh-pages
branch./docs
folder.Improve Navigation & UX
Add Versioning (Future Consideration)
Tasks
docs
(orbook
) folder.SUMMARY.md
.Additional Notes
If anyone has ideas for organization, content, or improvements to the structure, please comment below. Let’s build a user-friendly documentation and keep
autofirma-nix
growing!The text was updated successfully, but these errors were encountered: