Skip to content
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

fix(import): Fix relative import path error #2397

Merged
merged 2 commits into from May 9, 2024

Conversation

MortalHappiness
Copy link
Contributor

@MortalHappiness MortalHappiness commented May 8, 2024

Tracking issue

N/A

Why are the changes needed?

Running pyflyte run examples/basics/basics/documenting_workflows.py in the flytesnacks repo raises error.

image

What changes were proposed in this pull request?

Because documenting_workflows.py has relative imports in it, the additional_path should be its parent folder's parent folder.

How was this patch tested?

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
Signed-off-by: Kevin Su <pingsutw@apache.org>
@pingsutw pingsutw merged commit 94a48ae into flyteorg:master May 9, 2024
47 checks passed
austin362667 pushed a commit to austin362667/flytekit that referenced this pull request May 10, 2024
Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
Signed-off-by: Kevin Su <pingsutw@apache.org>
Co-authored-by: Kevin Su <pingsutw@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants