You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey @samber
Can I work on one of the tasks mentioned above (like adding the GitHub Action or config file)?
Just wanted to confirm if this issue is open for normal contributors or if it’s planned to be handled internally.
Hello, I wanted to work on this issue since it's been sitting for a while. I'm not sure how linting for code blocks should work. The steps I did were:
Add the mdsf config file, as per @samber's comment. Also specified golangci-lint as the binary while referencing its config file in the repo root
Running the commented mdsf command in the makefile: mdsf verify --debug --log-level warn docs/
This prints out a lot of false positives since the linter requires a full compilable go program, which the markdown snippets aren't. Example:
Am I missing something or is linting for snippets unfeasible?
Side note
The make tools command installs an old version of golangci-lint (1.xx) despite referencing @ latest. This conflicts with the linter config file in this repo, which requires version 2. I fixed this for me by specifying some 2.x version as per their docs. Should we align the makefile with this small adjustment?
Activity
intojhanurag commentedon Oct 5, 2025
Hey @samber
Can I work on one of the tasks mentioned above (like adding the GitHub Action or config file)?
Just wanted to confirm if this issue is open for normal contributors or if it’s planned to be handled internally.
samber commentedon Oct 5, 2025
This is open to contributors, but some improvements have to be made to mdsf before doing my checklist.
💯💯💯
intojhanurag commentedon Oct 5, 2025
Got it, thanks for clarifying! I’ll keep an eye on the mdsf updates and jump in once those improvements are ready.
ccoVeille commentedon Nov 12, 2025
The following PR brought most of what wascreporree as missing in this issue
samber commentedon Nov 12, 2025
Wonderful ✌️
@hougesen suggested the following config
{ "languages": { "go": { "binary": "/mylinter", "arguments": ["run", "--config", ".mylinter.yaml", "$PATH"] } } }AhmedCharfeddine commentedon May 8, 2026
Hello, I wanted to work on this issue since it's been sitting for a while. I'm not sure how linting for code blocks should work. The steps I did were:
golangci-lintas the binary while referencing its config file in the repo rootmdsf verify --debug --log-level warn docs/This prints out a lot of false positives since the linter requires a full compilable go program, which the markdown snippets aren't. Example:
Am I missing something or is linting for snippets unfeasible?
Side note
The
make toolscommand installs an old version ofgolangci-lint(1.xx) despite referencing @ latest. This conflicts with the linter config file in this repo, which requires version 2. I fixed this for me by specifying some 2.x version as per their docs. Should we align the makefile with this small adjustment?