-
Notifications
You must be signed in to change notification settings - Fork 45
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
Rectified the display name of BracketLocalBackend #92
Rectified the display name of BracketLocalBackend #92
Conversation
Thank you for this change @devilkiller-ag! Could you please add a unit test asserting this? |
Sure @kshitijc |
@kshitijc I added the test case |
@devilkiller-ag the tests are failing. Could you please fix? |
Sure, I am trying |
@kshitijc Is everything great now? |
Hi, @kshitijc , @IceKhan13, Is this PR ready to merge now? |
Sorry for the failure of previous styles check. I have rectified it and ran all the tests on local. It has been rated 10/10 now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect! Thank you for this contribution @devilkiller-ag! 🚀
Summary
I changed the value of
name
argument value passed to the super-class ofBraketLocalBackend
to solve the issue #87. The bug was due the value'sv_simulator'
passed in thename
argument instead of the backend name of called backend.