Skip to content
This repository was archived by the owner on Jul 17, 2018. It is now read-only.

Add hack to check node_modules exists #23

Merged
merged 2 commits into from
Feb 2, 2018
Merged

Conversation

NickColley
Copy link
Contributor

This is only needed because we are vendoring private govuk-frontend

This PR is done with a prototyping hat on, since we intend to delete this project once we've finished our private beta work, otherwise I would not be happy with this hack.

Let me know if this is OK.

As part of https://trello.com/c/mBYUk8Bx/680-fix-npm-install-warning-in-the-prototype-kit-beta

This is only needed because we are vendoring private govuk-frontend
@joelanman
Copy link
Contributor

nice, thanks. Wondering if we should start updating the Changelog with each PR, same as Fronted/Design system. Seems like a good approach

@NickColley
Copy link
Contributor Author

Updated changelog

@NickColley NickColley force-pushed the fix-npm-install-warning branch 2 times, most recently from 3d42fc1 to 6093723 Compare February 2, 2018 16:29
@NickColley NickColley force-pushed the fix-npm-install-warning branch from 6093723 to 34075c7 Compare February 2, 2018 16:31
Copy link
Contributor

@joelanman joelanman left a comment

Choose a reason for hiding this comment

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

Great work, thanks @NickColley

@NickColley NickColley merged commit 050dfb2 into master Feb 2, 2018
@NickColley NickColley deleted the fix-npm-install-warning branch February 2, 2018 16:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants