-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Rename __root_path__
to __development_base_path__
#9136
Conversation
To clarify that this path is not available in production installs (i.e. non-editable)
✅ Deploy Preview for prefect-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -33,7 +34,7 @@ async def ls(): | |||
List available recipes. | |||
""" | |||
|
|||
recipe_paths = Path(__file__).parent / ".." / "projects" / "recipes" | |||
recipe_paths = prefect.__module_path__ / "projects" / "recipes" |
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 don't have strong feelings about this change but it's consistent with the purpose of having the module path in the init.
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.
Makes sense to me!
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.
Nice
To clarify that this path is not available in production installs (i.e. non-editable) to avoid future problems like #9132 — this has bit me before as well.