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

Round 2 - August 2017 Dep updates #82

Merged
merged 2 commits into from
Aug 24, 2017

Conversation

zslabs
Copy link
Contributor

@zslabs zslabs commented Aug 18, 2017

✔︎ I've tested in all browsers (IE11, Firefox, Safari, Chrome, and iOS)

What should this PR do?

Round 2 of dependency updates this month. This round gets rid of those annoying warning messages we were seeing, and updates Webpack plus a few misc. packages.

There will be a RF PR to go with this as well which will have a new version of RS tagged in it once this is merged and released.

What is the acceptance criteria?

  1. Sans-errors

Copy link
Contributor

@BlakeGuilloud BlakeGuilloud left a comment

Choose a reason for hiding this comment

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

Sorry thought I had approved this already. We want this in for the code freeze?

@zslabs zslabs added the HOLD label Aug 22, 2017
@zslabs
Copy link
Contributor Author

zslabs commented Aug 22, 2017

@BlakeGuilloud Thanks man! Nah, no rush on this one - we'll probably just merge in after the release.

@zslabs zslabs removed the HOLD label Aug 24, 2017
…gust-dep-updates-2017-round2

# Conflicts:
#	build/scripts/rhinostyle-docs.js
#	dist/scripts/rhinostyle.min.js
#	package.json
#	yarn.lock
@zslabs zslabs merged commit 5e83244 into master Aug 24, 2017
@zslabs zslabs deleted the zach/august-dep-updates-2017-round2 branch August 24, 2017 16:03
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.

3 participants