Add tools/ dir and some handy utilities#702
Add tools/ dir and some handy utilities#702fdwr merged 15 commits intowebmachinelearning:mainfrom inexorabletash:tools
Conversation
The tools are intended for spec editors to use when authoring and/or reviewing changes. They aren't incorporated into the spec publishing process, but that could be considered in the future. - tools/reformat_js - reformats JS blocks in the spec - tools/lint.mjs - looks for convention violations and other glitches The reformat_js tools has been applied to index.bs
|
As long promised, here are some tools I've written to help with authoring the spec. They're inconsistent and best effort, but I've found them useful. I think they're too immature to incorporate into the Makefile (does anyone use that?) or the spec publishing action, but iteration is welcome! |
anssiko
left a comment
There was a problem hiding this comment.
Thanks a lot for these contributions!
I already test drove the lint tool and found it very helpful 👍
|
Also FYI I kicked off a thread with spec-prod https://lists.w3.org/Archives/Public/spec-prod/2024AprJun/0003.html - but let's start locally. |
fdwr
left a comment
There was a problem hiding this comment.
Thanks JB. 2 opinionated style thoughts, plus easy ones.
huningxin
left a comment
There was a problem hiding this comment.
These tools would be very useful! Thanks much @inexorabletash !
Co-authored-by: Ningxin Hu <[email protected]>
huningxin
left a comment
There was a problem hiding this comment.
Another nit: you may want to add npm related files into .gitignore:
/tools/node_modules/
/tools/package-lock.json
Done in 1a1e054 |
SHA: 74b9831 Reason: push, by fdwr Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
The tools are intended for spec editors to use when authoring and/or reviewing changes. They aren't incorporated into the spec publishing process, but that could be considered in the future.
The reformat-js.py tool has been applied to index.bs
Preview | Diff