Skip to content

Warning about removal of extras#10206

Closed
1ucian0 wants to merge 1 commit intoQiskit:mainfrom
1ucian0:remove-extras
Closed

Warning about removal of extras#10206
1ucian0 wants to merge 1 commit intoQiskit:mainfrom
1ucian0:remove-extras

Conversation

@1ucian0
Copy link
Copy Markdown
Member

@1ucian0 1ucian0 commented Jun 5, 2023

Qiskit 0.44 (with terra 0.25) needs to have an warning about the fact that Qiskit 0.45 (with terra 0.45) will stop having application as extras. This entry is more of a reminder about that.

@1ucian0 1ucian0 added this to the 0.25.0 milestone Jun 5, 2023
@qiskit-bot
Copy link
Copy Markdown
Collaborator

One or more of the the following people are requested to review this:

  • @Qiskit/terra-core

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 5178356488

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 16 unchanged lines in 4 files lost coverage.
  • Overall coverage increased (+0.06%) to 85.908%

Files with Coverage Reduction New Missed Lines %
crates/accelerate/src/sabre_swap/mod.rs 1 99.77%
crates/qasm2/src/expr.rs 1 93.76%
crates/qasm2/src/lex.rs 2 91.65%
crates/qasm2/src/parse.rs 12 97.11%
Totals Coverage Status
Change from base Build 5177933484: 0.06%
Covered Lines: 71215
Relevant Lines: 82897

💛 - Coveralls

@jakelishman
Copy link
Copy Markdown
Member

Let's leave this PR open as a reminder, but in practice we'll effectively need to completely rewrite the release note during the writing of the main prelude, so it might not make too much sense to actually merge it.

We also possibly need to discuss (if we haven't already) what extras we are going to make part of the qiskit package that aren't there yet - qiskit-terra has quite a few, some of which we might want to use this same opportunity to drop (with appropriate warnings).

@mtreinish mtreinish added the on hold Can not fix yet label Jul 17, 2023
@mtreinish
Copy link
Copy Markdown
Member

I've included the basic contents here in: Qiskit/qiskit-metapackage#1785 as part of the final metapackage release note. Starting in the next release Qiskit 0.45.0 we'll be doing everything from this repository.

@mtreinish mtreinish closed this Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

on hold Can not fix yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants