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
[FEATURE] Remove [cloud] optional dependency #9814
[FEATURE] Remove [cloud] optional dependency #9814
Conversation
@@ -1,2 +1 @@ | |||
great_expectations_cloud>=0.0.3.dev5 | |||
orjson>=3.9.7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Initially I tried to remove this whole file...but cloud_data_store.py needs orjson for the tests that are run in the develop
branch.
Keeping it in this branch, too, for consistency. (Even though CI passed without it.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Open question for me is if we need to release a new 0.18.x version after this PR goes in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Jira ticket: https://greatexpectations.atlassian.net/browse/Z2-289
Removing the optional [cloud] dependency so that we can pin the agent to a specific GX Core version (or at least a minimum version) in pyproject.toml rather than just poetry.lock.
Similar PR into
develop
is here: #9813This PR needs to be merged before we can merge great-expectations/cloud#229.
invoke lint
(usesruff format
+ruff check
)For more information about contributing, see Contribute.
After you submit your PR, keep the page open and monitor the statuses of the various checks made by our continuous integration process at the bottom of the page. Please fix any issues that come up and reach out on Slack if you need help. Thanks for contributing!