Skip to content

Conversation

@0candy
Copy link
Contributor

@0candy 0candy commented Jul 13, 2016

@0candy 0candy added this to the #Epic: Globalization milestone Jul 13, 2016
@0candy 0candy self-assigned this Jul 13, 2016
@0candy 0candy added the #review label Jul 13, 2016
@0candy 0candy force-pushed the initial_glob branch 3 times, most recently from 22e634e to 0b3bf18 Compare July 13, 2016 20:51
@0candy 0candy force-pushed the initial_glob branch 2 times, most recently from 3adb844 to 1eb6839 Compare July 27, 2016 15:53
@0candy
Copy link
Contributor Author

0candy commented Jul 27, 2016

@jannyHou or @superkhau PTAL. Thanks.

return cb(
new Error('Cannot connect to the data source.' +
' Ensure the configuration is valid and the connector is installed.'));
new Error(g.t('Cannot connect to the data source.' +
Copy link
Contributor

Choose a reason for hiding this comment

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

g.f

@jannyHou
Copy link
Contributor

@0candy Only some reminders that change g.t to g.f, they will be folded after change. Other than that LGTM 👍

@0candy 0candy force-pushed the initial_glob branch 2 times, most recently from 9bdc5d5 to ade210a Compare July 28, 2016 18:09
},
function createUser(next) {
console.log('Creating user %s with password %s', USER, PASSWORD);
console.log(g.f('Creating user %s with password %s', USER, PASSWORD));
Copy link
Contributor

Choose a reason for hiding this comment

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

I see inconsistency between using console.log or g.log

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 We should allow either but probably recommend g.log in globalization.md.

@0candy 0candy merged commit 2a7e072 into master Aug 3, 2016
@0candy 0candy deleted the initial_glob branch August 3, 2016 17:37
@0candy 0candy removed the #review label Aug 3, 2016
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.

4 participants