We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
storybook config from .percy.js is fully ignored even though it should be possible based on the docs: https://github.com/percy/percy-storybook/tree/master#percy-config-file-options
storybook
.percy.js
v20.9.0
@percy/cli
v1.26.3
@percy/storybook
v4.3.6
v4.3.7
it does work in v1.26.2
v1.26.2
yarn percy storybook ./build-storybook --verbose [percy] A new version of @percy/cli is available! 1.26.3 -> 1.27.3 [percy] Invalid config: <---- NOTICE THIS ⚠️ [percy] - storybook: unknown property <---- NOTICE THIS ⚠️ [percy:storybook] Requesting Storybook: http://localhost:65468 (0ms) [percy] Successfully downloaded Chromium 929475 [percy] Percy has started! [percy] Snapshot taken: src/accordion/AccordionItem: Default [percy] Snapshot taken: src/accordion/Accordion: Default [percy] Failed to create build [percy] Processing 381 snapshots...
yarn percy storybook ./build-storybook --verbose with .percy.js file:
yarn percy storybook ./build-storybook --verbose
module.exports = { version: 2, discovery: { concurrency: 2, // https://docs.percy.io/docs/debugging-sdks#assets-never-requested-by-asset-discovery networkIdleTimeout: 450 }, snapshot: { enableJavascript: true, minHeight: 900, percyCSS: 'iframe {display: none}', widths: [1024] }, storybook: { additionalSnapshots: [ { prefix: '[Dark mode] ', globals: { theme: 'dark' } } ], globals: { theme: 'light' }, include: ['src/accordion/Accordion: Default', 'src/announcement: Percy', 'src/button: Percy'] } };
yarn percy storybook ./build-storybook --verbose [percy] A new version of @percy/cli is available! 1.26.2 -> 1.27.3 [percy:config] Found config file: .percy.js (304ms) [percy:config] Using config: { version: 2, discovery: { concurrency: 2, networkIdleTimeout: 450 }, snapshot: { enableJavaScript: true, minHeight: 900, percyCSS: 'iframe {display: none}', widths: [ 1024 ] }, storybook: { additionalSnapshots: [ { prefix: '[Dark mode] ', globals: { theme: 'dark' } } ], globals: { theme: 'light' }, include: [ 'src/accordion/Accordion: Default', 'src/announcement: Percy', 'src/button: Percy' ] } } (18ms) [percy:storybook] Requesting Storybook: http://localhost:64593 (15ms) [percy:core:install] Successfully downloaded Chromium 929475 (44870ms) [percy:core:browser] Launching browser (0ms) [percy:core:browser] Browser connected [87314]: HeadlessChrome/96.0.4664.0 (1563ms) [percy:core:browser] Closing browser (0ms) [percy:core:browser] Browser closed (5ms)
The text was updated successfully, but these errors were encountered:
I'm using yarn as my package manager; and it ended up being an issue with internal percy dependencies not upgrading correctly.
yarn
Sorry, something went wrong.
No branches or pull requests
The problem
storybook
config from.percy.js
is fully ignored even though it should be possible based on the docs: https://github.com/percy/percy-storybook/tree/master#percy-config-file-optionsEnvironment
v20.9.0
but it doesn't matter@percy/cli
version:v1.26.3
where the issue starts; but it happens in all newer versions too@percy/storybook
version:v4.3.6
orv4.3.7
- fails in bothDetails
it does work in
v1.26.2
Debug logs
Code to reproduce issue
yarn percy storybook ./build-storybook --verbose
with.percy.js
file:output using
v1.26.2
✅The text was updated successfully, but these errors were encountered: