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

chore(docker and babel): no compile at runtime #54

Merged
merged 8 commits into from
Sep 10, 2019
Merged

Conversation

frickjack
Copy link
Contributor

  • compile at image build time - not at run time
  • do not use babel polyfills when not necessary

* compile at image build time
* do not use babel polyfills when not necessary
mfshao
mfshao previously approved these changes Sep 9, 2019
Copy link
Contributor

@mfshao mfshao left a comment

Choose a reason for hiding this comment

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

thanks!

@mfshao
Copy link
Contributor

mfshao commented Sep 10, 2019

https://github.com/uc-cdis/guppy/blob/master/stories/conf.js#L42 can you change this line to use port 80? otherwise the storybook won't display

Copy link
Contributor

@mfshao mfshao left a comment

Choose a reason for hiding this comment

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

awesome! thanks 👍

@frickjack frickjack merged commit f0d62e4 into master Sep 10, 2019
@frickjack frickjack deleted the fix/docker branch September 10, 2019 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants