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

Core: Fix features, core, logLevel in main.js config types #14745

Merged

Conversation

valerybugakov
Copy link
Contributor

Issue: features setting was added recently to the main.js config. It's not reflected in the StorybookConfig type.

What I did

Added features, core and loglevel to the StorybookConfig type.

How to test

Updated usage of StorybookConfig in examples should throw an error if the config type doesn't have some field.

@nx-cloud
Copy link

nx-cloud bot commented Apr 28, 2021

Nx Cloud Report

CI ran the following commands for commit 6b8c8b5. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status Command
#000000 nx run-many --target=prepare --all --parallel --max-parallel=15

Sent with 💌 from NxCloud.

@shilman shilman changed the title Add features to the StorybookConfig type Core: Add features to the StorybookConfig type Apr 28, 2021
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

@valerybugakov great addition! however I only see features added. Did you forget to check something in?

@valerybugakov
Copy link
Contributor Author

@shilman, right, I've pushed the last commit 👍

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

💯 !!!

@shilman shilman changed the title Core: Add features to the StorybookConfig type Core: Fix features, core, logLevel in main.js config types Apr 28, 2021
@shilman shilman merged commit 3179f64 into storybookjs:next Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants