storybookjs / eslint-plugin-storybook

🎗Official ESLint plugin for Storybook
MIT License
245 stars 52 forks source link

New Rule: Disallow template strings in titles #74

Closed IanVS closed 1 year ago

IanVS commented 2 years ago

Is your feature request related to a problem? Please describe. According to https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#string-literal-titles, dynamic titles are going to be prohibited in 7.0. It might be nice to have a linting rule that flags if backticks are being used.

Describe the solution you'd like A rule, not included in recommended (due to different versions of storybook having different requirements), which flags the use of backticks in story titles.

Additional context https://discord.com/channels/486522875931656193/691505730125168682/928685457552863254

image

YozhEzhi commented 1 year ago

recommended (due to different versions of storybook having different requirements)

I prefer the rule to be error. Because dynamic title is blocking using of "on demand feature".

IanVS commented 1 year ago

Closing in preference to https://github.com/storybookjs/eslint-plugin-storybook/issues/111