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(build): run Netlify for components package #2406

Merged
merged 2 commits into from
May 8, 2019

Conversation

asudoh
Copy link
Contributor

@asudoh asudoh commented May 7, 2019

We have some nice ideas wrt how to enable multiple Netlify previews per package. This PR is a stop-gap approach until we make the idea in action - Which is, build deploy artifacts for each and collect them into a single directory so the index.html can refer to.

Changelog

New

  • Code to build demo and copy them to a build directory.
  • package.json script for above.

Changed

  • Demo code so it works in non-root directory.

Testing / Reviewing

Testing should make sure our demo is not broken, especially regular dev env.

@asudoh asudoh force-pushed the netlify-components branch from ba06a50 to c391506 Compare May 7, 2019 07:14
@netlify
Copy link

netlify bot commented May 7, 2019

Deploy preview for the-carbon-components ready!

Built with commit 7546d3f

https://deploy-preview-2406--the-carbon-components.netlify.com

@asudoh asudoh force-pushed the netlify-components branch from c391506 to 3b51b1e Compare May 7, 2019 08:27
@asudoh asudoh force-pushed the netlify-components branch from 3b51b1e to 1ffbb9d Compare May 7, 2019 08:38
@asudoh asudoh marked this pull request as ready for review May 7, 2019 08:50
Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

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

tested locally, looks good to me!

Copy link
Member

@alisonjoseph alisonjoseph left a comment

Choose a reason for hiding this comment

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

Demo looks good 👍

@asudoh asudoh merged commit ee65576 into carbon-design-system:master May 8, 2019
@asudoh asudoh deleted the netlify-components branch May 8, 2019 02:09
@asudoh
Copy link
Contributor Author

asudoh commented May 8, 2019

Thanks @emyarod and @alisonjoseph for reviewing!

asudoh added a commit to asudoh/carbon-components that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs carbon-design-system#2406.
asudoh added a commit to asudoh/carbon-components that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs carbon-design-system#2406.
asudoh added a commit to asudoh/carbon-components that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs carbon-design-system#2406.
asudoh added a commit to asudoh/carbon-components that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs carbon-design-system#2406.
asudoh added a commit to asudoh/carbon-components that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs carbon-design-system#2406.
asudoh added a commit that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs #2406.
asudoh added a commit that referenced this pull request May 9, 2019
GitHub workflow stuffs we are investigating may solve this problem
space. This PR is a stop-gap approach until we make the idea in action
- Which is, build deploy artifacts for each and collect them into a
single directory so the index.html can refer to.

This also enables Netlify build for our React variant.

Refs #2406.
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.

3 participants