Skip to content

Globalize loopback-sdk-angular#230

Merged
davidcheung merged 1 commit intomasterfrom
globalize
Aug 4, 2016
Merged

Globalize loopback-sdk-angular#230
davidcheung merged 1 commit intomasterfrom
globalize

Conversation

@davidcheung
Copy link
Copy Markdown
Contributor

@davidcheung davidcheung commented Jul 21, 2016

@davidcheung
Copy link
Copy Markdown
Contributor Author

@slnode test please

Comment thread .gitignore
loopback-sdk-angular-*.tgz
!intl/
intl/*
!intl/en/
Copy link
Copy Markdown
Member

@Amir-61 Amir-61 Jul 27, 2016

Choose a reason for hiding this comment

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

  • Add an empty line at the end of file;
  • I think you don't need !intl/, do you?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

there is an empty line EOF, or else github gives you an icon

@0candy
Copy link
Copy Markdown
Contributor

0candy commented Aug 4, 2016

LGTM. Please squash commits and merge.

@davidcheung
Copy link
Copy Markdown
Contributor Author

@slnode test please

@davidcheung davidcheung merged commit 0caea05 into master Aug 4, 2016
@davidcheung davidcheung deleted the globalize branch August 4, 2016 21:14

/* eslint quotes: ["error", "single"] */
var fs = require('fs');
var g = require('strong-globalize')();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should be inside the try/catch below where require('loopback') is done. This is what allows apidocs.strongloop.com to work without installing any dependencies.

This line is the root cause of strongloop-internal/apidocs.strongloop.com#56 :-(

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@rmg thanks for catching this!

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