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

CA-172263: Hide base-vdi post SXM between FileSR's and LVHDSR's #267

Closed
wants to merge 1 commit into from

Conversation

siddharthv
Copy link
Contributor

Signed-off-by: Siddharth Vinothkumar siddharth.vinothkumar@citrix.com

@siddharthv
Copy link
Contributor Author

@germanop Could you please expedite the review on this one ?
This fix has to be delivered urgently.

Thanks

@siddharthv
Copy link
Contributor Author

The travis job was successful.
Its coveralls that's causing the problem!

@germanop
Copy link
Contributor

Issued with the wrong author and signed-off-by

@germanop germanop closed this Jul 14, 2015
@germanop germanop reopened this Jul 14, 2015
@germanop
Copy link
Contributor

I am gonna accept this patch anyway.

@siddharthv
Copy link
Contributor Author

@germanop Apologies, I'll add them to the commit message...
Could you please let me know what tag you would like me to add ?

@germanop
Copy link
Contributor

That's fine. Just add Acked-by

Signed-off-by: Germano Percossi <germano.percossi@citrix.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants