Skip to content

Conversation

@jzempel
Copy link
Member

@jzempel jzempel commented Oct 22, 2024

Description

This PR adds missing demo directories 😬 to eslint.

Detail

A less disruptive version of #1956. I learned that strategies could be selectively applied for eslint --fix autofixing – resulting in a lower impact PR. Material changes are:

@jzempel jzempel requested a review from a team as a code owner October 22, 2024 17:59
@jzempel jzempel changed the title Jzempel/demo lint @jzempel fix: add storybook demos to eslint config Oct 22, 2024
@coveralls
Copy link

Coverage Status

coverage: 95.896%. remained the same
when pulling c92ac40 on jzempel/demo-lint
into bbe1263 on main.

Copy link
Contributor

@ze-flo ze-flo left a comment

Choose a reason for hiding this comment

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

I like this better! 💯

@jzempel jzempel changed the title @jzempel fix: add storybook demos to eslint config fix: add storybook demos to eslint config Oct 22, 2024
@jzempel jzempel merged commit 821d0f2 into main Oct 22, 2024
9 checks passed
@jzempel jzempel deleted the jzempel/demo-lint branch October 22, 2024 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants