Skip to content

Remove legacy providers interface#361

Merged
manoelmarques merged 2 commits into
qiskit-community:mainfrom
manoelmarques:interface
Apr 20, 2022
Merged

Remove legacy providers interface#361
manoelmarques merged 2 commits into
qiskit-community:mainfrom
manoelmarques:interface

Conversation

@manoelmarques
Copy link
Copy Markdown
Contributor

Summary

See Qiskit/qiskit#7886

Details and comments

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 12, 2022

Pull Request Test Coverage Report for Build 2173265187

  • 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.989%

Totals Coverage Status
Change from base Build 2173258613: 0.0%
Covered Lines: 4059
Relevant Lines: 4461

💛 - Coveralls

@woodsp-ibm
Copy link
Copy Markdown
Member

We should probably have a release note for this.

t-imamichi
t-imamichi previously approved these changes Apr 13, 2022
@manoelmarques
Copy link
Copy Markdown
Contributor Author

Terra has a release note but I will add one here too

@manoelmarques
Copy link
Copy Markdown
Contributor Author

Terra removed the legacy interfaces from its code and now the build fails. This PR needs to be merged.

@manoelmarques manoelmarques merged commit e412c49 into qiskit-community:main Apr 20, 2022
@manoelmarques manoelmarques deleted the interface branch April 20, 2022 12:56
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.

4 participants