search

medikoo / cli-color

Colors and formatting for the console
ISC License
676 stars 34 forks source link

Corrected package.json "main" key #5

Closed statico closed 11 years ago

statico commented 11 years ago

Not listing lib/index.js explicitly as the module's main library has caused me trouble. (Node v0.8.14)

medikoo commented 11 years ago

@statico can you specify what exactly is the issue? require('cli-color') works totally fine on v0.8.14

statico commented 11 years ago

Yep, that works. But we have a few private, local modules, and one of them depended on cli-color, and when we tried to require our module, the include failed. This patch fixed that.

medikoo commented 11 years ago

Can you provide more details? e.g. what is the relative path ofcli-color from local module that couldn't import it. Sorry for being picky, but I need to understand the exact issue, and prefer not to take pull requestes blindly. Cheers!

statico commented 11 years ago

Sorry, I don't have time for a test case. It's fine with me if you want to close this.

medikoo commented 11 years ago

Ok. I assume problem lied elsewhere. Closed