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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃彈 Run default task in parallel with storybook-amp #30392

Merged
merged 8 commits into from
Sep 25, 2020

Conversation

alanorozco
Copy link
Member

Previously, you'd have to run gulp at the same time as storybook-amp to serve binaries from localhost:8000.

@amp-owners-bot
Copy link

Hey @wassgha! These files were changed:

build-system/tasks/storybook/index.js

@google-cla google-cla bot added the cla: yes label Sep 24, 2020
build-system/tasks/storybook/index.js Outdated Show resolved Hide resolved
@dvoytenko
Copy link
Contributor

LGTM for me, but please also get LGTM from @ampproject/wg-infra

build-system/tasks/serve.js Outdated Show resolved Hide resolved
build-system/tasks/storybook/index.js Outdated Show resolved Hide resolved
build-system/tasks/storybook/index.js Outdated Show resolved Hide resolved
build-system/tasks/storybook/index.js Outdated Show resolved Hide resolved
@alanorozco alanorozco merged commit c075697 into ampproject:master Sep 25, 2020
@alanorozco alanorozco deleted the gulp-storybook branch September 25, 2020 17:34
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
Previously, you'd have to run gulp at the same time as storybook-amp to serve binaries from localhost:8000.
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

5 participants