Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Conversation

@prafulVaishnav
Copy link
Contributor

No description provided.

@prafulVaishnav
Copy link
Contributor Author

@nethip Can you please look into this PR

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: formatting

@nethip
Copy link
Contributor

nethip commented Jul 24, 2015

@prafulVaishnav Do we need to update our unit tests with the new changes?

@nethip
Copy link
Contributor

nethip commented Jul 24, 2015

LGTM. Please run the unit tests and let me know if they are passing, so that I can go ahead with the merge.

@prafulVaishnav
Copy link
Contributor Author

@nethip Unit Test Cases are working fine..

nethip added a commit that referenced this pull request Jul 24, 2015
Checking the registry if theme is present and then logging it.
@nethip nethip merged commit 00123c4 into release Jul 24, 2015
@marcelgerber marcelgerber deleted the prafulVaishnav/fixIssue branch May 23, 2016 07:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants