-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Fixing bug in qcstyle
that modified custom style dict
#11874
Conversation
Thank you for opening a new pull request. Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone. One or more of the the following people are requested to review this:
|
Pull Request Test Coverage Report for Build 8082344327Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
qcstyle
that modified custom style dictqcstyle
that modified custom style dict
releasenotes/notes/qcstyle-bug-custom-style-dicts-22deab6c602ccd6a.yaml
Outdated
Show resolved
Hide resolved
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.
LGTM thanks for the fix!
* fixed qcstyle bug that modified style dict, added test * black * ensure that the keys in nested_attrs are not overwritten again * release notes * Update releasenotes/notes/qcstyle-bug-custom-style-dicts-22deab6c602ccd6a.yaml --------- Co-authored-by: Julien Gacon <gaconju@gmail.com> (cherry picked from commit d21127a)
) * fixed qcstyle bug that modified style dict, added test * black * ensure that the keys in nested_attrs are not overwritten again * release notes * Update releasenotes/notes/qcstyle-bug-custom-style-dicts-22deab6c602ccd6a.yaml --------- Co-authored-by: Julien Gacon <gaconju@gmail.com> (cherry picked from commit d21127a) Co-authored-by: Alexander Miessen <52243504+Durd3nT@users.noreply.github.com>
Summary
visualization.circuit.qcstyle
popped elements from the style dictionary instead of extracting them, resulting in modyfied style dicts.As discussed with @Cryoris, this PR solves this issue, leaving style dicts untouchted.
Details and comments
Solved the issue described above, added test.