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

Delete the re-imports in the consts.py module from the apify-shared #163

Open
vdusek opened this issue Nov 16, 2023 · 0 comments
Open

Delete the re-imports in the consts.py module from the apify-shared #163

vdusek opened this issue Nov 16, 2023 · 0 comments
Labels
debt Code quality improvement or decrease of technical debt. t-tooling Issues with this label are in the ownership of the tooling team.

Comments

@vdusek
Copy link
Contributor

vdusek commented Nov 16, 2023

We have introduced apify-shared-python to consolidate general constants for both the Client and SDK.

To facilitate a seamless transition to the new package, we have implemented re-imports with DeprecationWarning in the existing codebase - consts.py.

This is only a temporary state and after a few new releases, we should get rid of it.
@vdusek vdusek added t-tooling Issues with this label are in the ownership of the tooling team. debt Code quality improvement or decrease of technical debt. labels Nov 16, 2023
@vdusek vdusek self-assigned this Nov 20, 2023
@vdusek vdusek removed their assignment Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
debt Code quality improvement or decrease of technical debt. t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vdusek