feat: Add module_doc_lint to CI - #1707
jstoobysmith wants to merge 10 commits into
Conversation
|
Thank you for this pull-request (PR). If this is your first PR, welcome to the community! Below is what will happen next. Please read carefully if you are not familiar with the process. You may open other PRs while this one is being reviewed, and can stack PRs on top of each other, so don't let these steps slow you down.
Tip: The easiest way to get have a fast review is to submit a PR that is small and self-contained, and has clear documentation explaining why things are the way they are in your chages. If you have any problems or questions, please reach out to the community on the Zulip. |
|
Looks good, except I think we should remove the |
|
@morrison-daniel Implemented this change. -awaiting-author |
morrison-daniel
left a comment
There was a problem hiding this comment.
It looks like linters fail now since some new files were added, but otherwise looks good. I'll leave my approval and you can merge after fixing the linters.
Made with the help of Opus 5.5.
Look at the individual commits here rather than the overall files changed, as they tell a better story. The idea here is to: