Skip to content

doesn't remove lodash import given it's still used in the example #1411

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

Merged
merged 6 commits into from
Jul 14, 2017

Conversation

olistik
Copy link
Contributor

@olistik olistik commented Jul 11, 2017

No description provided.

@TheDutchCoder
Copy link
Collaborator

Good catch, but maybe I'm missing something, but I don't see the change to lodash?

If you could add the import in, then we should be good I think.

@olistik
Copy link
Contributor Author

olistik commented Jul 11, 2017

@TheDutchCoder you're right. A Diff's Diff is not a joy to handle :-D
Let me know if the import has been correctly restored by my latest commit.

@TheDutchCoder
Copy link
Collaborator

Fixed the whitespace, rest looks good ;)

Now hoping Travis works (there's been an issue with React components linting it seems)

@TheDutchCoder TheDutchCoder requested review from skipjack and removed request for skipjack July 14, 2017 00:40
Copy link
Collaborator

@TheDutchCoder TheDutchCoder left a comment

Choose a reason for hiding this comment

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

Good to go!

@skipjack skipjack merged commit 1a942ab into webpack:master Jul 14, 2017
@olistik olistik deleted the patch-1 branch July 14, 2017 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants