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

Bump github.com/cpuguy83/go-md2man/v2 to v2.0.1 #1321

Merged
merged 1 commit into from Jan 4, 2022

Conversation

anthonyfok
Copy link
Contributor

@anthonyfok anthonyfok commented Jan 3, 2022

What type of PR is this?

  • bug
  • cleanup
  • documentation
  • feature

What this PR does / why we need it:

github.com/cpuguy83/go-md2man/v2 v2.0.1 has been released with fixes, especially the following change which affects man page output:

It is good to bump our dependency on go-md2man to an official release from an old snapshot. For example, GitHub CLI https://github.com/cli/cl bumped dependency on go-md2man to v2.0.1 back in October 2021.

Which issue(s) this PR fixes:

None (no filed GitHub issue).

However, this PR fixes build/test error in Debian package golang-github-urfave-cli when built with go-md2man v2.0.1, failing at TestToMan, see:

  1. List of recent CI tests (v1): https://ci.debian.net/packages/g/golang-github-urfave-cli/testing/amd64/
  2. List of recent CI tests (v2): https://ci.debian.net/packages/g/golang-github-urfave-cli-v2/testing/amd64/

Release Notes

Bump github.com/cpuguy83/go-md2man/v2 to v2.0.1

Also fix test expectation for man contents: dash and underscore
are no longer escaped with backslash in go-md2man v2.0.1

See cpuguy83/go-md2man#74
@rliebz rliebz merged commit c2cf7ed into urfave:master Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants