Skip to content

Commit

Permalink
🏗 Update Storybook forbidden terms (ampproject#35991)
Browse files Browse the repository at this point in the history
1. Move `withA11y` entry to `forbiddenTermsGlobal`. It wasn't targetting the right files previously.

2. Add `@storybook/addon-knobs` since it's deprecated.

Partial for ampproject#35923
  • Loading branch information
alanorozco authored and Mahir committed Sep 9, 2021
1 parent 052f1d4 commit 5b319f2
Showing 1 changed file with 49 additions and 2 deletions.
51 changes: 49 additions & 2 deletions build-system/test-configs/forbidden-terms.js
Original file line number Diff line number Diff line change
Expand Up @@ -726,6 +726,55 @@ const forbiddenTermsGlobal = {
],
checkInTestFolder: true,
},
'withA11y':
'The Storybook decorator "withA11y" has been deprecated. You may simply remove it, since the a11y addon is now globally configured.',
'@storybook/addon-knobs': {
message:
'The @storybook/addon-knobs package has been deprecated. Use Controls instead (`args` and `argTypes`). https://storybook.js.org/docs/react/essentials/controls',
allowlist: [
// TODO(#35923): Update existing files to use Controls instead.
'build-system/tasks/storybook/amp-env/main.js',
'build-system/tasks/storybook/preact-env/main.js',
'extensions/amp-animation/0.1/storybook/template.js',
'extensions/amp-date-display/1.0/storybook/Basic.amp.js',
'extensions/amp-date-display/1.0/storybook/Basic.js',
'extensions/amp-iframe/0.1/storybook/Basic.amp.js',
'extensions/amp-iframe/1.0/storybook/Basic.amp.js',
'extensions/amp-image-slider/0.1/storybook/Basic.amp.js',
'extensions/amp-inline-gallery/1.0/storybook/Basic.js',
'extensions/amp-lightbox/1.0/storybook/Basic.amp.js',
'extensions/amp-lightbox/1.0/storybook/Basic.js',
'extensions/amp-lightbox-gallery/1.0/storybook/Basic.amp.js',
'extensions/amp-lightbox-gallery/1.0/storybook/Basic.js',
'extensions/amp-render/1.0/storybook/Basic.js',
'extensions/amp-selector/1.0/storybook/Basic.amp.js',
'extensions/amp-selector/1.0/storybook/Basic.js',
'extensions/amp-sidebar/0.1/storybook/Basic.amp.js',
'extensions/amp-sidebar/1.0/storybook/Basic.amp.js',
'extensions/amp-sidebar/1.0/storybook/Basic.js',
'extensions/amp-soundcloud/1.0/storybook/Basic.amp.js',
'extensions/amp-soundcloud/1.0/storybook/Basic.js',
'extensions/amp-stream-gallery/1.0/storybook/Basic.amp.js',
'extensions/amp-stream-gallery/1.0/storybook/Basic.js',
'extensions/amp-timeago/1.0/storybook/Basic.js',
'extensions/amp-twitter/0.1/storybook/Basic.amp.js',
'extensions/amp-twitter/1.0/storybook/Basic.amp.js',
'extensions/amp-twitter/1.0/storybook/Basic.js',
'extensions/amp-video/1.0/storybook/Basic.amp.js',
'extensions/amp-video/1.0/storybook/Basic.js',
'extensions/amp-video-iframe/1.0/storybook/Basic.amp.js',
'extensions/amp-vimeo/1.0/storybook/Basic.amp.js',
'extensions/amp-vimeo/1.0/storybook/Basic.js',
'extensions/amp-wordpress-embed/1.0/storybook/Basic.amp.js',
'extensions/amp-youtube/0.1/storybook/Basic.amp.js',
'extensions/amp-youtube/1.0/storybook/Basic.amp.js',
'extensions/amp-youtube/1.0/storybook/Basic.js',
'src/builtins/storybook/amp-layout-with-aspect-ratio-css.amp.js',
'src/builtins/storybook/amp-layout.amp.js',
'src/preact/storybook/Context.js',
'src/preact/storybook/Wrappers.js',
],
},
};

const bannedTermsHelpString =
Expand Down Expand Up @@ -1047,8 +1096,6 @@ const forbiddenTermsSrcInclusive = {
'third_party/react-dates/scope-require.js',
],
},
'withA11y':
'The Storybook decorator "withA11y" has been deprecated. You may simply remove it, since the a11y addon is now globally configured.',
'detectIsAutoplaySupported': {
message:
'Detecting autoplay support is expensive. Use the cached function "isAutoplaySupported" instead.',
Expand Down

0 comments on commit 5b319f2

Please sign in to comment.