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

Update environment_setup.md #1129

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update environment_setup.md #1129

wants to merge 3 commits into from

Conversation

ssen85
Copy link

@ssen85 ssen85 commented Sep 18, 2023

The function name in the code snippet is mentioned as my_wf() whereas the actual workflow name is hello_world_wf().

The code does not compile with the actual example in the page. NameError: name 'my_wf' is not defined

Signed-off-by: ssen85 shubhabrata.sen@gmail.com

The function name in the code snippet is mentioned as my_wf() whereas the actual workflow name is hello_world_wf().

The code does not compile with the actual example in the page.
NameError: name 'my_wf' is not defined

Signed-off-by: ssen85 <shubhabrata.sen@gmail.com>

Signed-off-by: ssen85 <shubhabrata.sen@gmail.com>
@ssen85
Copy link
Author

ssen85 commented Sep 27, 2023

Hi, do I need to make any further changes to this pull request for it to be merged ?

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