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

CommonJS && AMD support #30

Merged
merged 3 commits into from May 10, 2014
Merged

Conversation

alessioalex
Copy link
Contributor

So we can use it with Browserify and Require.js for example

@adamschwartz
Copy link
Owner

Thanks @alessioalex. Looks great. Please bump the package.json version as well and I'll merge.

@alessioalex
Copy link
Contributor Author

@adamschwartz I did it. Btw can you also publish it to NPM? I couldn't find it: https://www.npmjs.org/package/log-with-style

adamschwartz added a commit that referenced this pull request May 10, 2014
@adamschwartz adamschwartz merged commit 4ffcd21 into adamschwartz:master May 10, 2014
@adamschwartz
Copy link
Owner

Done. Although I accidentally pushed as hs instead of adamschwartz. Trying to get that resolved now but it's there at least.

@adamschwartz
Copy link
Owner

Okiedokie. All set: https://www.npmjs.org/package/log-with-style. Thanks again @alessioalex!

@alessioalex
Copy link
Contributor Author

@adamschwartz thank you for your awesome library!

Btw can you please also add the repository to the package.json file (so people can find this repo from the NPM registry)? An example: https://github.com/visionmedia/express/blob/master/package.json#L83

@adamschwartz
Copy link
Owner

Done

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