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.

Sign up for GitHub

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

Jump to bottom

API: deprecate legacy API #612

Closed
martinfleis opened this issue Jun 18, 2024 · 1 comment · Fixed by #619
Closed

API: deprecate legacy API #612

martinfleis opened this issue Jun 18, 2024 · 1 comment · Fixed by #619
Assignees
@martinfleis
Labels
api deprecation
Milestone
0.8.0

Comments

@martinfleis
Copy link
Member

We need to properly use the decorators to depreciate all of the Legacy API.

On top, we should probably add either env. variable or some global setting allowing folks to silence all of those warnings, otherwise a typical workflow would be very noisy.
@martinfleis martinfleis added this to the 0.8.0 milestone Jun 18, 2024
@martinfleis martinfleis self-assigned this Jun 18, 2024
@jGaboardi
Copy link
Member

xref #617

@martinfleis martinfleis mentioned this issue Jun 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martinfleis martinfleis

Labels
api deprecation
Projects
None yet
Milestone
0.8.0
Development

Successfully merging a pull request may close this issue.

API: deprecate legacy API in favour of Graph-based functional implementation martinfleis/momepy
2 participants
@jGaboardi @martinfleis