Skip to content

Build demo into dist folder#80

Merged
jamesrweb merged 6 commits intoP5-wrapper:masterfrom
yevdyko:chore/build-demo-into-dist-folder
Mar 30, 2021
Merged

Build demo into dist folder#80
jamesrweb merged 6 commits intoP5-wrapper:masterfrom
yevdyko:chore/build-demo-into-dist-folder

Conversation

@yevdyko
Copy link
Contributor

@yevdyko yevdyko commented Mar 29, 2021

This MR updates the Webpack config to build a demo into the dist folder.

What was changed?

  • Format Webpack config.
  • Build demo into the dist folder.
  • Update deploy script with a new build directory path.
  • Remove example build directory from ignore files.
  • Use space indent in the .editorconfig.
  • Prefer to use double quotes.

@yevdyko
Copy link
Contributor Author

yevdyko commented Mar 29, 2021

@jamesrweb Could you test the deploy script?

@yevdyko yevdyko changed the title Build Demo Into Dist Folder Build Demo into Dist Folder Mar 29, 2021
Copy link
Member

@jamesrweb jamesrweb left a comment

Choose a reason for hiding this comment

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

Looks good to me, just 2 minor points to change and I can merge once changed! Thanks for the awesome work man, you're on fire!👍🏻

@yevdyko yevdyko requested a review from jamesrweb March 30, 2021 08:59
@jamesrweb jamesrweb merged commit 8199acf into P5-wrapper:master Mar 30, 2021
@yevdyko yevdyko deleted the chore/build-demo-into-dist-folder branch March 30, 2021 12:57
@yevdyko yevdyko changed the title Build Demo into Dist Folder Build demo into dist folder May 13, 2021
jamesrweb pushed a commit that referenced this pull request Aug 15, 2022
* Format webpack config

* Build demo into dist folder

* Update deploy script with new build directory path

* Remove example build directory from ignore files

* Use space indent in the .editorconfig

* Use double quotes
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