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

docs: generate with eslint-doc-generator #275

Merged
merged 1 commit into from Nov 8, 2022

Conversation

bmish
Copy link
Contributor

@bmish bmish commented Nov 5, 2022

What:

Add eslint-doc-generator and update docs with it.

Why:

eslint-doc-generator is a CLI tool I built for automating the generation of the README rules list table and rule doc title/notices for ESLint plugins. It follows common documentation conventions from this and other top ESLint plugins and will help us standardize documentation across ESLint plugins (and generally improve the usability of custom rules through better documentation and streamline the process of adding new rules). It has 100% test coverage and is used in many of the top ESLint plugins already (including eslint-plugin-jest).

How:

  • Replace generate-readme-table script
  • Add lint:generate-readme-table to run on CI

Checklist:

  • Documentation
  • Tests - N/A
  • Ready to be merged

@G-Rath G-Rath enabled auto-merge (squash) November 5, 2022 17:22
@codecov
Copy link

codecov bot commented Nov 5, 2022

Codecov Report

Merging #275 (58c699f) into main (e1d8511) will not change coverage.
The diff coverage is n/a.

❗ Current head 58c699f differs from pull request most recent head 7485ab6. Consider uploading reports for the commit 7485ab6 to get more accurate results

@@            Coverage Diff            @@
##              main      #275   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines          575       575           
  Branches       165       165           
=========================================
  Hits           575       575           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

auto-merge was automatically disabled November 7, 2022 16:29

Head branch was pushed to by a user without write access

@@ -47,6 +47,10 @@ jobs:
if: ${{ matrix.eslint == 8 }}
run: npm run validate

- name: ▶️ Ensure docs are up-to-date
if: ${{ matrix.eslint == 8 && matrix.node == 16 }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Attempting to fix CI by ensuring this check only runs on the latest Node version. No need to run on earlier versions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can someone approve CI to run?

@G-Rath G-Rath enabled auto-merge (squash) November 8, 2022 01:04
@G-Rath G-Rath changed the title Automate docs with eslint-doc-generator docs: generate with eslint-doc-generator Nov 8, 2022
@G-Rath G-Rath disabled auto-merge November 8, 2022 01:11
@G-Rath G-Rath merged commit 4299024 into testing-library:main Nov 8, 2022
@github-actions
Copy link

🎉 This PR is included in version 4.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

4 participants