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

[patch] Switch to chalk #3680

Merged
merged 1 commit into from
Apr 4, 2016
Merged

[patch] Switch to chalk #3680

merged 1 commit into from
Apr 4, 2016

Conversation

markelog
Copy link
Contributor

@markelog markelog commented Apr 1, 2016

Not sure if you want test with that
Fixes #3668

@sailsbot
Copy link

sailsbot commented Apr 1, 2016

Thanks for posting, @markelog! We'll look into this ASAP.

@mikermcneil
Copy link
Member

Thanks @markelog!

sails.log.info(('To shut down Sails, press <CTRL> + C at any time.'));
sails.log.blank();
sails.log('--------------------------------------------------------'.grey);
sails.log((':: ' + new Date()).grey);
sails.log();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a heads up-- I believe this changes the logs. (Probably fine, but just wanted to point it out)

@mikermcneil mikermcneil added the patch This label is deprecated. Please don't use it anymore label Apr 1, 2016
@markelog markelog force-pushed the chalk branch 5 times, most recently from bd7eb83 to a0d9254 Compare April 2, 2016 18:52
@mikermcneil mikermcneil merged commit e04a6a0 into balderdashy:master Apr 4, 2016
@mikermcneil
Copy link
Member

@markelog thanks again! And especially for taking the time to follow up to make it easier for me to merge 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch This label is deprecated. Please don't use it anymore
Development

Successfully merging this pull request may close these issues.

Undefined output with NODE_ENV=production
3 participants