-
Notifications
You must be signed in to change notification settings - Fork 38
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
Feature Request: Incorporating the blogs into official documentation #188
Comments
Thanks for the suggestion and the feedback on the blog posts! I think 1 or 3 would make sense (I don’t think having the documentation "externally" is really a good practice, indeed).
A lot of questions to be answered before choosing one strategy over the other :) Btw, the source of the blog post are in markdown and available at https://github.com/didrocks/website (more specifically https://github.com/didrocks/website/tree/master/content/blog) |
@didrocks the fact that the source of the blogs are in markdown makes everything even easier (I think) I'll take a stab at incorporating the text into the code, and send a PR when I'm done (might start with one or two, at first) |
OK, I got a draft with a raw dump of the What's New. Here it is: https://github.com/Lockszmith/zsys-fork-for-pr/blob/documentation/cmd/zsysd/client/docs.go |
Great! Thanks for working on it! I just had a look, some feedbacks:
import _ "embed"
//go:embed what-new.man
var whatNewContent string And then, we can simply
What do you think? |
All of the points are ones I've been thinking about as well. Sub-commands, as I see it, there are 2 options:
I love the embedded solution, I didn't know about it, and from what you're writing it seems it's not available yet, but yes, that's exactly what we should do, I'll read up on that now. Noted about the buffer, and totally agree on the paging. This troubled me as well, I think I'll use |
On numbering, I think if we can sightely rephrase the content to be mainconcepts, taking also the bionic posts (I guess the others are fine as more "per topic") and that way, this is less shocking :) The embedded solution is going to be ready with Go 1.16, which should be out in the coming days, so, let’s just be patient :) Ack on using more, I wonder just if there is a go package to deal with that directly instead of redirecting the output outself… Needs investigation :) |
Just to keep track: https://stackoverflow.com/questions/28705716/paging-output-from-go |
Pager used, detection with a fuzzy decision on activating it. Link is still: https://github.com/Lockszmith/zsys-fork-for-pr/blob/documentation/cmd/zsysd/client/docs.go |
Excellent! Happy for you that you go into go (pun intended) universe a little bit more :) I had a quite look at your current work, generally, it seems to be exactly what is needed, some comments:
As an additional bonus, I think the general "doc" command should (without any argument) display all available documentation (and ofc, documentation needs to be available in the completion). On rendering text into console, it really depends on what you want to do, if this is only some bold/color work, I suggest using |
yes, I'm doing all of this in a docker container, and since I wanted to test out the embedding, I just went ahead and installed go 1.16 via:
And I'm building now with:
I'll take a look at using |
TIL from this issue that zsys is documented in a series of blog posts. I noted in #207 that I have no idea what zsys is trying to do, in detail. Until now, all I had seen was the command-usage stuff in the manpage and repo README - nothing about intent or objectives or approach. While I agree with this issue that these docs should migrate to a better home in the repo, to be updated as further development occurs.. In the meantime, can we PLEASE have a set of links to the blog posts at the top of the README, so that at least they are discoverable. I'm off to go read now, but I'd have been much happier to have found these when they were written or when I came here to look previously. |
That’s a good one, added to the README. Thanks |
The Blog series at https://didrocks.fr/ is excellent.
I hope on of the suggestions would be agreeable with you:
Have the README.md and the wiki Home-Page (or just disable the Wiki) point to the blog's tag, so that everything will be referenced from here - I only found about the blog site from the issues.
I am willing to take the task upon myself (you already did all the heavy lifting of writing the content) and put the content wherever you want, just choose the approach that suites you, and I'll be happy to accommodate.
If #1 is the approach you want to take, keep in mind GitHub doesn't provide a PR workflow for wiki.
Let me know if I can help.
The text was updated successfully, but these errors were encountered: