Skip to content
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

Deprecate extensions on stable/0.46 #11498

Merged
merged 7 commits into from
Jan 9, 2024

Conversation

Cryoris
Copy link
Contributor

@Cryoris Cryoris commented Jan 5, 2024

Summary

Deprecation analogue of #11488 for the stable/0.46 branch.

@Cryoris Cryoris added Changelog: Deprecation Include in "Deprecated" section of changelog Changelog: Removal Include in the Removed section of the changelog labels Jan 5, 2024
@Cryoris Cryoris requested review from nonhermitian and a team as code owners January 5, 2024 15:45
@qiskit-bot
Copy link
Collaborator

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

  • @enavarro51
  • @Qiskit/terra-core
  • @mtreinish
  • @nkanazawa1989

@Cryoris Cryoris added this to the 0.46.0 milestone Jan 5, 2024
Copy link
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this. The biggest comment is that 0.46 shouldn't contain any new removals (it's a warning-only release), and most of the ones that have been effected here are were only deprecated in 0.45, so they wouldn't have been valid for removal anyway.

edit: I didn't highlight every removal in the review, but all of them need reverting.

... that still need one more release for removal
@Cryoris
Copy link
Contributor Author

Cryoris commented Jan 8, 2024

I reverted the eager removals in a31325a 🙂

@Cryoris Cryoris removed the Changelog: Removal Include in the Removed section of the changelog label Jan 8, 2024
jakelishman
jakelishman previously approved these changes Jan 8, 2024
Copy link
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good now, thanks - one comment that I missed before, if you feel like updating it.

jakelishman
jakelishman previously approved these changes Jan 8, 2024
Copy link
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thanks!

@jakelishman jakelishman enabled auto-merge January 8, 2024 15:09
Copy link
Contributor

@ElePT ElePT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving to return to the queue.

@jakelishman jakelishman added this pull request to the merge queue Jan 9, 2024
Merged via the queue into Qiskit:stable/0.46 with commit 5e16db2 Jan 9, 2024
12 checks passed
@Cryoris Cryoris deleted the deprecate-extensions/0.46 branch January 9, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Deprecation Include in "Deprecated" section of changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants