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

Updates node-form-data dependency, 0.1.3->0.2.0, to fix webpack issue. #616

Merged
merged 1 commit into from Apr 5, 2015

Conversation

wali-s
Copy link
Contributor

@wali-s wali-s commented Apr 4, 2015

SuperAgent depends on node-form-data@0.1.3, which uses node-mime@1.3.4 which has issues with webpack on the server-side (Ref: #615).

node-form-data@0.2.0 uses another library, mime-types@2.0.3, which does not have these issues.

@defunctzombie
Copy link
Contributor

Needs to depend on an exact version down to the patch level.

What effect does this have on the bundle size?

SuperAgent depends on node-form-data@0.1.3, which uses node-mime@1.3.4 which has issues with webpack on the server-side (Ref: ladjs#615).

node-form-data@0.2.0 uses another library, mime-types@2.0.3, which does not have these issues.
@wali-s wali-s force-pushed the update-form-data-dependency branch from 00cd4b5 to 10ae2c8 Compare April 4, 2015 22:49
@wali-s
Copy link
Contributor Author

wali-s commented Apr 4, 2015

Fixed the version issue.

If you mean my webpack bundle, in my particular case, the bundle size is unchanged. I believe because both node-mime and mime-types were already included as dependencies for other libraries.

Details on a related issue that may be fixed by this PR (browserify support): form-data/form-data#95

defunctzombie added a commit that referenced this pull request Apr 5, 2015
Updates node-form-data dependency, 0.1.3->0.2.0, to fix webpack issue.
@defunctzombie defunctzombie merged commit 1b1f315 into ladjs:master Apr 5, 2015
@wali-s wali-s deleted the update-form-data-dependency branch April 5, 2015 03:36
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.

None yet

2 participants