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

Update to docker setup procedures #46

Merged
merged 1 commit into from
May 26, 2017
Merged

Update to docker setup procedures #46

merged 1 commit into from
May 26, 2017

Conversation

corviday
Copy link
Contributor

Last month, webpack was updated to patch a security vulnerability involving DNS rebinding. Added directions on how to configure webpack to accept requests addressed to the docker server's address.

Copy link
Contributor

@jameshiebert jameshiebert left a comment

Choose a reason for hiding this comment

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

Great, this looks like a very helpful update to the docs. Thanks for adding it!

@@ -59,13 +59,46 @@ You can lint all files `npm run lint`, or a specific file `npm run lint:glob <fi

Use the `git/hooks/pre-commit-eslint` (and install into your .git/hooks directory) to abort a commit if any staged `*.js` files fail linting (warnings OK).

If you *realy* want to skip the linting during a commit, you can always run `git commit --no-verify`. However, this is not recommended.
If you *really* want to skip the linting during a commit, you can always run `git commit --no-verify`. However, this is not recommended.
Copy link
Contributor

Choose a reason for hiding this comment

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

Bah, thanks. Good catch :)

@jameshiebert jameshiebert merged commit 4d15c29 into master May 26, 2017
@jameshiebert jameshiebert deleted the doc-update branch May 26, 2017 22:05
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