-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[MAINTENANCE] Update default action list in Checkpoint
based on user environment
#8074
Conversation
✅ Deploy Preview for niobium-lead-7998 canceled.
|
@@ -2118,6 +2118,8 @@ def _resolve_add_checkpoint_args( # noqa: PLR0913 | |||
"Must either pass in an existing checkpoint or individual constructor arguments (but not both)" | |||
) | |||
|
|||
action_list = action_list or self._determine_default_action_list() |
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.
I'd love to push this down into the Checkpoint
but that would require an isinstance(context, CloudDataContext)
. Thoughts? Feels a little bad here.
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.
Perhaps another selling point for CloudCheckpoint
?
…_expectations into m/lakitu-190/update_default_action_list
…_expectations into m/lakitu-190/update_default_action_list
…hub.com/great-expectations/great_expectations into m/lakitu-190/update_default_action_list
…ons/great_expectations into m/_/pandas2-only * 'm/_/pandas2-only' of https://github.com/great-expectations/great_expectations: [MAINTENANCE] Add public_api to read_parquet (#8060) [BUGFIX] fix issue-7954 (#7963) [MAINTENANCE] Delete `LegacyCheckpoint` (#8082) [BUGFIX] Fix broken Cloud tests around `Checkpoint` (#8083) [MAINTENANCE] Update PR template to reference `invoke` for linting (#8079) [MAINTENANCE] Ruff `0.0.271` update (#8077) [MAINTENANCE] Update default action list in `Checkpoint` based on user environment (#8074)
…r environment (#8074) Co-authored-by: Rob Lim <robert.m.lim@gmail.com>
Cloud contexts should not have reference to
UpdateDataDocs
Description of PR changes above includes a link to an existing GitHub issue
PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB]
Code is linted
Appropriate tests and docs have been updated
For more details, see our Contribution Checklist, Coding style guide, and Documentation style guide.
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!