Skip to content

Conversation

mtreinish
Copy link
Member

Summary

This commit removes the pkgutil namespace hooks that are used for extending the qiskit.* and qiskit.providers.* namespaces from external packages. These were previously used to enable the elements packaging model where different parts of qiskit lived in separate python packages under a shared namespace. This is no longer being used and leaving the namespace extendable by external packages carries a risk of issues or other accidental cross-interactions when an external package gets loaded as part of Qiskit.

Details and comments

This commit removes the pkgutil namespace hooks that are used for
extending the `qiskit.*` and `qiskit.providers.*` namespaces from
external packages. These were previously used to enable the elements
packaging model where different parts of qiskit lived in separate python
packages under a shared namespace. This is no longer being used and
leaving the namespace extendable by external packages carries a risk of
issues or other accidental cross-interactions when an external package
gets loaded as part of Qiskit.
@mtreinish mtreinish added the Changelog: Removal Include in the Removed section of the changelog label Nov 9, 2023
@mtreinish mtreinish added this to the 1.0.0pre1 milestone Nov 9, 2023
@mtreinish mtreinish requested review from jyu00 and a team as code owners November 9, 2023 14:47
@qiskit-bot
Copy link
Collaborator

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

  • @Qiskit/terra-core

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.

🎉

@jakelishman jakelishman enabled auto-merge November 9, 2023 14:53
@coveralls
Copy link

Pull Request Test Coverage Report for Build 6813409425

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 32 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.03%) to 86.927%

Files with Coverage Reduction New Missed Lines %
qiskit/pulse/library/waveform.py 3 93.75%
crates/qasm2/src/lex.rs 5 92.17%
crates/qasm2/src/parse.rs 24 96.2%
Totals Coverage Status
Change from base Build 6801884624: -0.03%
Covered Lines: 74331
Relevant Lines: 85510

💛 - Coveralls

@jakelishman jakelishman added this pull request to the merge queue Nov 9, 2023
Merged via the queue into Qiskit:main with commit dab572a Nov 9, 2023
@mtreinish mtreinish deleted the remove-namespace-extensions branch November 9, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Removal Include in the Removed section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants