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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change Renderer interface to satisfy go lint #240
Comments
It's a valid issue, but I'm afraid it might be a non-starter because it'd invalidate all implementations of the interface, which is a huge breaking API change. It's definitely not worth the cost to fix 2 minor lint issues. I think this is an issue that should be resolved with the |
Version 2 sounds good to me 馃憤 |
golint is not part of the tests since our Markdown dependency defines an interface that is not lint-compliant (unfortunately). See russross/blackfriday#240
This issue can be closed I believe. |
Confirmed, no longer an issue in https://goreportcard.com/report/gopkg.in/russross/blackfriday.v2#golint Closing. |
I know this sounds a bit pedantic, but the
Renderer
interface has two methods thatgolint
complains about:BlockHtml
andRawHtmlTag
. It suggestsBlockHTML
andRawHTMLTag
.FWIW I'm working to improve our score on Go Report Card... 馃槃
The text was updated successfully, but these errors were encountered: