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

Fix missing text in "man gh-codespace-ssh" #6368

Merged
merged 1 commit into from Sep 29, 2022

Conversation

anthonyfok
Copy link
Contributor

@anthonyfok anthonyfok commented Sep 28, 2022

Rearrange the long help text so that '--config' does not appear at the start of a line which groff interprets as an undefined macro and skips (i.e. not renders) the content of that whole line:

'--config' option that generates per-codespace ssh configuration in OpenSSH

To reproduce:

$ go run cmd/gen-docs/main.go --man-page --doc-path /tmp/gh-manpages
$ man --warnings -E UTF-8 -l -Tutf8 -Z /tmp/gh-manpages/gh-codespace-ssh.1 >/dev/null
troff: <standard input>:26: warning: macro '--config'' not defined

Discovered by the Debian Lintian tool.

Rearrange the long help text so that '--config' does not appear at the
start of a line which groff interprets as an undefined macro and skips
the content of that whole line.

To reproduce:

  $ go run cmd/gen-docs/main.go --man-page --doc-path /tmp/gh-manpages
  $ man --warnings -E UTF-8 -l -Tutf8 -Z /tmp/gh-manpages/gh-codespace-ssh.1 >/dev/null
  troff: <standard input>:26: warning: macro '--config'' not defined

Discovered by the Debian Lintian tool.
@anthonyfok anthonyfok requested a review from a team as a code owner Sep 28, 2022
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Sep 28, 2022
@cliAutomation
Copy link
Collaborator

cliAutomation commented Sep 28, 2022

Hi! Thanks for the pull request. Please ensure that this change is linked to an issue by mentioning an issue number in the description of the pull request. If this pull request would close the issue, please put the word 'Fixes' before the issue number somewhere in the pull request body. If this is a tiny change like fixing a typo, feel free to ignore this message.

@cliAutomation cliAutomation added this to Needs review 🤔 in The GitHub CLI Sep 28, 2022
@anthonyfok anthonyfok changed the title Fix missing text in gh-codespace-ssh.1 man page Fix missing text in "man gh-codespace-ssh" Sep 28, 2022
mislav
mislav approved these changes Sep 29, 2022
Copy link
Member

@mislav mislav left a comment

Thanks for the workaround!

For a long-term fix, I'd like us to report this and fix it in the upstream md2man library which is supposed to be escaping these things. This issue might be related cpuguy83/go-md2man#46

The GitHub CLI automation moved this from Needs review 🤔 to Needs to be merged 🎉 Sep 29, 2022
@mislav mislav merged commit 8fc120d into cli:trunk Sep 29, 2022
7 checks passed
The GitHub CLI automation moved this from Needs to be merged 🎉 to Pending Release 🥚 Sep 29, 2022
@github-actions github-actions bot moved this from Pending Release 🥚 to Done 💤 in The GitHub CLI Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external pull request originating outside of the CLI core team
Projects
The GitHub CLI
  
Done 💤
Development

Successfully merging this pull request may close these issues.

None yet

3 participants