Skip to content

Commit

Permalink
Merge pull request #1547 from PrefectHQ/doc_tweak
Browse files Browse the repository at this point in the history
Remove pitfall doc reference to DockerEnvironment
  • Loading branch information
joshmeek authored Sep 24, 2019
2 parents 3092363 + 1bfd080 commit 745d4ff
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/core/concepts/common-pitfalls.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ is_serializable(my_flow) # returns True / False
Note that this is _not_ a guarantee, but merely a helper to catch issues early on.

::: tip
A good rule of thumb is that every object / function you rely on should either be explicitly attached to some attribute of the flow (e.g., a task) or importable via some library that is included in your `DockerEnvironment` requirements.
A good rule of thumb is that every object / function you rely on should either be explicitly attached to some attribute of the flow (e.g., a task) or importable via some library that is included in your `Docker` storage requirements.
:::

A more robust (albeit longer) process to check that your Flow makes it through the serialization and deployment is to build its `DockerEnvironment` locally and check that the Flow can be deserialized within the container. For details on how to do this, see the corresponding section in the [local debugging](../tutorials/local-debugging.html#locally-check-your-flow-s-environment) tutorial.
A more robust (albeit longer) process to check that your Flow makes it through the serialization and deployment is to build its `Docker` storage locally and check that the Flow can be deserialized within the container. For details on how to do this, see the corresponding section in the [local debugging](../tutorials/local-debugging.html#locally-check-your-flow-s-docker-storage) tutorial.

0 comments on commit 745d4ff

Please sign in to comment.