Stripped BOM before parsing package.json #11

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@BorisKozo
Contributor

BorisKozo commented Nov 11, 2012

I copied the method from node (as we discussed on IRC).
Now npm can parse a package.json file that has BOM (for example when created by Visual Studio)

Added a test.
Made sure the test fails without the code change.

@BorisKozo

This comment has been minimized.

Show comment
Hide comment
@BorisKozo

BorisKozo Nov 26, 2012

Contributor

Ping

Contributor

BorisKozo commented Nov 26, 2012

Ping

@isaacs

This comment has been minimized.

Show comment
Hide comment
@isaacs

isaacs Nov 26, 2012

Member

Landed on 0.1.9. Thanks!

Member

isaacs commented Nov 26, 2012

Landed on 0.1.9. Thanks!

@isaacs isaacs closed this Nov 26, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment