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

Partial revert of 075841971b603bbe5774880ee1fa6abf778b84b0 #390

Merged
merged 1 commit into from
Apr 13, 2024

Conversation

tobixen
Copy link
Member

@tobixen tobixen commented Apr 13, 2024

An "import *" was removed in above commit. Yes, "import *" is bad practice - but since those things are reimported other places, we can't remove it in a minor release. First we need some kind of deprecation notice, then we need to remove it when doing a major release. I aim for every major release to be backward compatible with everything that wasn't marked as deprecated in the previous x.0.0 major release, so 3.0 is the earliest we can remove this.

An "import *" was removed in above commit.  Yes, "import *" is bad
practice - but since those things are reimported other places, we
can't remove it in a minor release.  First we need some kind of
deprecation notice, then we need to remove it when doing a major
release.  I aim for every major release to be backward compatible
with everything that wasn't marked as deprecated in the previous
x.0.0 major release, so 3.0 is the earliest we can remove this.
@tobixen tobixen force-pushed the backward-compatibility-fix branch from 5a9e6fe to b324044 Compare April 13, 2024 16:17
@tobixen tobixen added this pull request to the merge queue Apr 13, 2024
Merged via the queue into master with commit 6c424e7 Apr 13, 2024
8 checks passed
@tobixen tobixen deleted the backward-compatibility-fix branch April 13, 2024 16:27
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.

1 participant