search

nodejs / node

Node.js JavaScript runtime ✨🐢🚀✨
https://nodejs.org
Other
104.51k stars 28.19k forks source link

build: support `lint-md-fix` #53024

Closed RedYetiDev closed 1 week ago

RedYetiDev commented 2 weeks ago

Similar to lint-js-fix, this PR adds submit for lint-md-fix, which supplies the --fix CLI argument to the Markdown linter.

RedYetiDev commented 2 weeks ago

@nodejs/linting @nodejs/build-files

nodejs-github-bot commented 2 weeks ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59248/

lpinca commented 2 weeks ago

~Can you also add it to vcbuild.bat?~

Nvm, lint-js-fix is also not supported with vcbuild.bat.

RedYetiDev commented 2 weeks ago

~Can you also add it to vcbuild.bat?~

Nvm, lint-js-fix is also not supported with vcbuild.bat.

Once this lands, I can try to open a second PR to add them both (if it is possible)

RedYetiDev commented 1 week ago

So, I feel pretty stupid right now :laughing: the markdown linter doesn't support a --fix flag, I must be crazy, because I could've sworn it did.