-
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
Environment Dependency Healthcheck #1653
Conversation
There are a couple of changelog commits because I had some wonky changelog "conflicts" |
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.
Changelog request, otherwise great implementation!
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.
awesome! our deploys are going to get a whole lot better with this + @wagoodman 's work!
Thanks for contributing to Prefect!
Please describe your work and make sure your PR:
CHANGELOG.md
(if appropriate)docs/outline.toml
for API reference docs (if appropriate)Note that your PR will not be reviewed unless all three boxes are checked.
What does this PR change?
Adds a new healthcheck for environment specific dependencies. These are exposed through the
.dependencies
property on environmentsCloses #1623
Why is this PR important?
This allows us to remove some required dependencies from the docker storage build step and is a great notifier for users of certain environments