Skip to content

V3 mods rebase test #4

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

Closed
wants to merge 3 commits into from
Closed

V3 mods rebase test #4

wants to merge 3 commits into from

Conversation

JamieDixon
Copy link

No description provided.

Modify prod build

Move package over to @connected-home

Downgrade jest and babel-jest to support V3

Undo commented template section. Should work now that were referencing react-scripts as part of CRA rather than a folder in our project

Delete yarn.lock file and use npm for this proj instead

Remove fs package that was added by us for some reason

Add HivehomeWebappFaviconsWebpackPlugin with proper config

Remove path package from devDependencies
Adjust a few bits so that Jest works properly with V3

Add property to jest config for matching jest files in this lower version of jest
…the end

Bump favicon plugin ref to 2.0.0

Change favicon config to use ios rather than iphone key

Bump versions
@JamieDixon JamieDixon force-pushed the v3-mods-rebase-test branch from 4d56a8a to 600e989 Compare August 17, 2017 07:45
@sandfox
Copy link

sandfox commented Aug 17, 2017

looks dope

@JamieDixon
Copy link
Author

Thanks @sandfox
I've done the same rebase on #1 and merged. Cheers for looking this over.

@JamieDixon JamieDixon closed this Aug 17, 2017
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