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

refactor(cleanup): remove deprecated methods and properties #8701

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

gforsyth
Copy link
Member

@gforsyth gforsyth commented Mar 19, 2024

op as a method on another operation was deprecated in 4.0.
output_dtype and output_shape were deprecated in 7.0.

I think 2 major versions is a reasonable deprecation cycle.

BREAKING CHANGE: Deprecated methods and properties op, output_dtype, and output_shape are removed. op is no longer needed, and use .dtype and .shape respectively for the other two.

`op` as a method on another operation was deprecated in 4.0.
`output_dtype` and `output_shape` were deprecated in 7.0.

I think 2 major versions is a reasonable deprecation cycle.
@kszucs
Copy link
Member

kszucs commented Mar 20, 2024

I am feeling nostalgic :)

@cpcloud cpcloud added this to the 9.0 milestone Mar 20, 2024
@cpcloud cpcloud added the breaking change Changes that introduce an API break at any level label Mar 20, 2024
@cpcloud cpcloud changed the title chore(cleanup): remove deprecated methods and properties refactor(cleanup): remove deprecated methods and properties Mar 20, 2024
@cpcloud cpcloud merged commit 90c5a86 into ibis-project:main Mar 20, 2024
85 checks passed
@cpcloud cpcloud added the refactor Issues or PRs related to refactoring the codebase label Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Changes that introduce an API break at any level refactor Issues or PRs related to refactoring the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants