Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automate docs with eslint-doc-generator #242

Closed
wants to merge 1 commit into from

Conversation

bmish
Copy link
Member

@bmish bmish commented Oct 9, 2022

I built this CLI tool eslint-doc-generator for automating the generation of the README rules list and the title/notices in each rule doc. It follows common documentation conventions from this and other top ESLint plugins. The goal is to improve the usability of our custom rules through better documentation and make it easier to add new rules by automating more of the process. I was also able to remove a documentation generator script and tests that are now redundant.

@bmish bmish added the docs label Oct 9, 2022
@bmish
Copy link
Member Author

bmish commented Oct 9, 2022

Looks like this is blocked until we drop Node 12 support (#222).

Copy link
Collaborator

@platinumazure platinumazure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see why we couldn't proceed with this. All we would need to do is not run the lint:docs step on Node 12 or lower in CI. In package.json, you're only adding devDependencies, so the end user experience shouldn't be affected. I have no problem requiring a higher version of Node to develop this plugin than to run it.

Am I missing something?

@bmish
Copy link
Member Author

bmish commented Oct 12, 2022

@platinumazure good point, I'll try to update CI to support that in my new PR #243.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants