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

Revert "Make this lib peerDepend on loopback-connector" #1336

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

ssh24
Copy link
Contributor

@ssh24 ssh24 commented Apr 19, 2017

Revert back #1326 due to loopback requiring loopback-connector as a default dependency through this module.

connect to strongloop/loopback-workspace#487

@ssh24 ssh24 self-assigned this Apr 19, 2017
Make lib peerDepend on loopback-connector (#1326)

This reverts commit 67e8f37.
@ssh24 ssh24 force-pushed the revert-1326-fix-connector-dependency branch from 04a8d29 to c1d8397 Compare April 19, 2017 17:33
@ssh24 ssh24 requested a review from raymondfeng April 19, 2017 17:33
@ssh24
Copy link
Contributor Author

ssh24 commented Apr 19, 2017

@slnode test please

Copy link
Contributor

@jannyHou jannyHou left a comment

Choose a reason for hiding this comment

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

LGTM but cassandra fails

Copy link
Contributor

@loay loay left a comment

Choose a reason for hiding this comment

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

LGTM
Just make sure that cassandra tests pass before merging

@ssh24 ssh24 merged commit bcc5b55 into master Apr 19, 2017
@ssh24 ssh24 deleted the revert-1326-fix-connector-dependency branch April 19, 2017 18:14
@ssh24 ssh24 added apex and removed needs-priority labels Apr 19, 2017
@ssh24 ssh24 added this to the Sprint 34 - Apex milestone Apr 19, 2017
kjdelisle pushed a commit that referenced this pull request May 2, 2017
 * create sequence for nosql id (#1354) (Janny)
 * Fix order of query results (Loay)
 * Add DateString type (Kevin Delisle)
 * datatype.test: use predefined date (Kevin Delisle)
 * Update api documents (Loay)
 * Datasource documentation tune-up (Kevin Delisle)
 * Added unit tests specific to DateType where null (#1349) (Andrew McDonnell)
 * Fix/geo null (#1334) (paulussup)
 * replace exception thrown for invalid dates (Diana Lau)
 * Revert PR #1326 (#1336) (Sakib Hasan)
 * Make lib peerDepend on loopback-connector (#1326) (Russ Tyndall)
 * Add test case using updateAttributes (Loay)
 * Fix forceId bug for updateOrCreate (Loay)
 * Fix typo in description (jannyHou)
 * Fix relations test case (loay)
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