Skip to content

Refactor discovery models #212

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

Merged
merged 1 commit into from
Feb 28, 2017
Merged

Refactor discovery models #212

merged 1 commit into from
Feb 28, 2017

Conversation

loay
Copy link
Contributor

@loay loay commented Feb 14, 2017

Description

Refactor discovery test cases

Related issues

connect to loopbackio/loopback-connector#15

@loay loay added the review label Feb 14, 2017
@loay loay changed the title Refactor discovery models [DonRefactor discovery models Feb 14, 2017
@loay loay changed the title [DonRefactor discovery models [Don't Merge - WIP] Refactor discovery models Feb 14, 2017
@loay loay removed the review label Feb 15, 2017
@loay loay force-pushed the refactor-discovery branch from 91568d4 to c43235a Compare February 22, 2017 19:35
@loay
Copy link
Contributor Author

loay commented Feb 28, 2017

@slnode test please

@loay loay force-pushed the refactor-discovery branch from 223c326 to 115ce24 Compare February 28, 2017 19:28
@loay
Copy link
Contributor Author

loay commented Feb 28, 2017

@slnode test please

1 similar comment
@loay
Copy link
Contributor Author

loay commented Feb 28, 2017

@slnode test please

@loay loay changed the title [Don't Merge - WIP] Refactor discovery models Refactor discovery models Feb 28, 2017
Copy link
Contributor

@raymondfeng raymondfeng left a comment

Choose a reason for hiding this comment

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

LGTM

@loay loay force-pushed the refactor-discovery branch from 6946569 to c6922c8 Compare February 28, 2017 21:44
@loay loay merged commit edf05de into master Feb 28, 2017
@loay loay deleted the refactor-discovery branch February 28, 2017 21:49
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.

3 participants