Skip to content

Conversation

@Setogit
Copy link

@Setogit Setogit commented Jan 13, 2016

connect to strongloop-internal/scrum-nodeops#1157

index.js Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

alphabetize requires

Copy link
Author

Choose a reason for hiding this comment

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

done.

@sam-github sam-github assigned Setogit and unassigned sam-github Jan 14, 2016
@sam-github
Copy link
Contributor

@Setogit you did a search-n-replace for console, but you don't protect words that must not be translated. Won't that mean that the translated output is wrong?

@Setogit
Copy link
Author

Setogit commented Jan 15, 2016

Added some more {{ }} translation protection.

@Setogit
Copy link
Author

Setogit commented Jan 16, 2016

@sl-node test please

@Setogit
Copy link
Author

Setogit commented Jan 24, 2016

@sl-note test please

- node_modules is created with node v5
- globalize git.js
@Setogit Setogit assigned sam-github and unassigned Setogit Jan 25, 2016
@Setogit
Copy link
Author

Setogit commented Jan 25, 2016

dependant: strong-pm build failed. Two other dependants strong-arc and strongloop succeeded.

@Setogit
Copy link
Author

Setogit commented Jan 26, 2016

This requires strongloop/strong-globalize#5

Copy link
Contributor

Choose a reason for hiding this comment

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

please keep requires alphabetized

@sam-github
Copy link
Contributor

LGTM

_don't merge until strong-globalize is published_

@sam-github sam-github assigned Setogit and unassigned sam-github Jan 27, 2016
@Setogit Setogit removed their assignment Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants