Skip to content

Rename run_options to options in subclasses#8720

Closed
ikkoham wants to merge 6 commits into
Qiskit:mainfrom
ikkoham:primitives/rename-run_options2
Closed

Rename run_options to options in subclasses#8720
ikkoham wants to merge 6 commits into
Qiskit:mainfrom
ikkoham:primitives/rename-run_options2

Conversation

@ikkoham
Copy link
Copy Markdown
Contributor

@ikkoham ikkoham commented Sep 9, 2022

Summary

Alternative of #8719
(@ikkoham's suggestion)

Details and comments

@ikkoham ikkoham requested a review from jyu00 September 9, 2022 05:10
@qiskit-bot
Copy link
Copy Markdown
Collaborator

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:

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 9, 2022

Pull Request Test Coverage Report for Build 3023064427

  • 12 of 12 (100.0%) changed or added relevant lines in 6 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.003%) to 84.306%

Files with Coverage Reduction New Missed Lines %
qiskit/quantum_info/operators/mixins/linear.py 2 90.91%
Totals Coverage Status
Change from base Build 3017902091: 0.003%
Covered Lines: 57833
Relevant Lines: 68599

💛 - Coveralls

@ikkoham
Copy link
Copy Markdown
Contributor Author

ikkoham commented Sep 12, 2022

We adopted #8719.

@ikkoham ikkoham closed this Sep 12, 2022
@ikkoham ikkoham reopened this Sep 14, 2022
@ikkoham ikkoham closed this Sep 14, 2022
@ikkoham ikkoham deleted the primitives/rename-run_options2 branch September 14, 2022 04:09
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