-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Conversation
@omajid, @nguerrera, @mmitche fyi |
Seems like "skip ci please" from PR instructions has no effect 🙈 |
Were you able to launch these new man pages and confirm that they look good? Was there no manual fixup needed? |
Thank you for providing the steps used. |
This is awesome! 👍 I ran
I think we need some pre-processing of the .md files or post-processing of man pages. |
Can you add a Edit: Here is how it looks after the Some things that still look wrong:
|
@omajid fixed. I checked |
(also updated the bash script in top post and powershell script in comment with |
For the formatting issues, should we instead try fixing them or reporting them upstream: https://github.com/jgm/pandoc to make most profit for world (instead of manual fixes)? |
Pandoc seems to have extension points using custom filters http://pandoc.org/filters.html, that can be written in ruby, perl or python and use it as |
For 2.1, maybe we just fix up the formatting manually? Pains me to say that, but I don't want this to derail into getting nothing done for 2.1. |
After playing around a bit, I think I have something decent working, without any manual modifications (but see the filter file below). This what This is the command I used to generate the man pages:
I know the The filter file (a giant hack!) is: https://gist.github.com/omajid/22c4bbfdd6a8e235b2248f168f8523d0 This was done with: $ pandoc --version |
@nguerrera, my two cents: merge this PR as these files were last updated two years ago, so this changeset is already much better than what we have in master. Subsequent improvements can be done in follow up PRs and we can think of how to automate this process in #7291. |
I might have been convinced but @omajid's pages look so nice. 😁 Here's what I'd like:
Can you do that? I'd say we can last several releases if we're down to one button to update the man pages with a script. Thanks so much to both of you for pushing on this! |
@kasper3 feel free to update this PR and include the filter file. Could you please add a .NET Foundation license header to it as well? |
@omajid, thanks for the filter script. @nguerrera, I have added tool scripts in a separate directory and update the docs. There is a readme in tools dir with usage example https://github.com/dotnet/cli/blob/ee24d81648b6792c7249c5044f6218f59b89e49c/Documentation/manpages/tool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@omajid, a lot has improved in their core since then. I have updated readme with note to promote using latest version of pandoc under Ubuntu example, how to quickly install it from packages available for different distros at https://github.com/jgm/pandoc/releases. For other distros, it can be built using tarball https://github.com/jgm/pandoc/blob/master/INSTALL.md#linux. |
That's a good point. In fact, looking at the release notes, it seems like one of the issues I was seeing (bad links) was fixed in 2.0.5: "omit internal links (#4136). That is, just print the link text without the URL.". And thanks again everyone for getting this resolved before 2.1! |
skip ci please