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

Disable AMD for json3 #1005

Merged
merged 1 commit into from
Oct 21, 2016
Merged

Conversation

davidmatas
Copy link
Contributor

@davidmatas davidmatas commented Oct 13, 2016

Saying to Webpack don't use the AMD style for json3 dependency

Fix #1004

Saying to Webpack don't use the AMD style for
json3 dependency
@darrachequesne
Copy link
Member

@davidmatas thanks for the report! Could you confirm that this is the only failing dependency?

@davidmatas
Copy link
Contributor Author

davidmatas commented Oct 19, 2016

@darrachequesne Yes, in my tests this is the only one. I've tested a new build with the new webpack config and works fine.

What about the CI failing check? It should not be related :/

@darrachequesne
Copy link
Member

@davidmatas no it's not related, iOS 7.1 is now deprecated. I'll remove it.

@darrachequesne darrachequesne merged commit 5077f0a into socketio:master Oct 21, 2016
@darrachequesne
Copy link
Member

Thanks!

@darrachequesne darrachequesne added this to the 1.5.1 milestone Oct 24, 2016
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