-
Notifications
You must be signed in to change notification settings - Fork 79
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
refactor readme #37
Comments
I'll add that I'm pro flat README 😁 I think the Table of Contents works great. |
Just to add an extra opinion. I found it relatively simple to convert the single README to an ebook as a little reference for on my e-Reader, only needed minor post-processing to fix the code blocks and add a ToC using Calibre, the "#" headers become chapters and sections etc. Relative image URLs would help a bit too but not a big deal. |
Love the flat README - as is it's pretty brief on each topic I think, so still hitting the README feel. If you start adding multiple examples to each section could see what you mean about breaking it out. If you want a bit of both, you can setup github actions that do fancier processing on doc commits. For example with https://github.com/newaetech/chipwhisperer-target-cw308t we have a GH action that copies all the |
I would love to have a downloadable, self-contained PDF or EPUB that can be viewed offline. |
Pandoc is pretty good at converting markdown into a readable PDF/EPUB. I don't like the idea of keeping generated binary PDF/EPUB files in the repository, but perhaps this is something we could include in releases like we do with the schematic PDFs. |
Agree with @fharding1, i like including a pdf/epub in the release bundles. We're overdue to release the v1.1 bundle, but i have a few readme more additions i should include before we roll that. |
Previously considered refactoring the readme into separate readmes per interface, but have gotten lots of feedback on the 'flat' readme format, so will keep it.
I added a toc in 1e7971c (first generated with doctoc, but manually tweaked for brevity - it should be rather static so no need to automate updates)
Originally the 'usage' was supposed to be broken down by header, but became a list of protocols instead.
Then, the 'pinouts' was supposed to be a raw pinout list, but became a protocol specific list instead.
The text was updated successfully, but these errors were encountered: