-
Notifications
You must be signed in to change notification settings - Fork 15
refactor: rename input parameters to flowInput and deps for clarity #557
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
base: 12-23-feat_asymmetric_handler_signatures_for_step_array_methods
Are you sure you want to change the base?
Conversation
|
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
View your CI Pipeline Execution ↗ for commit 6090a08
☁️ Nx Cloud last updated this comment at |
1ed5f8d to
6090a08
Compare
🔍 Preview Deployment: Website✅ Deployment successful! 🔗 Preview URL: https://pr-557.pgflow.pages.dev 📝 Details:
_Last updated: _ |

Rename input parameters in step handlers for better clarity
This PR renames the input parameters in step handlers to improve clarity and better reflect their purpose:
flowInputinstead ofinput.runto directly access flow inputdepsinstead ofinputto access outputs from dependency stepsctxwhere usedThese changes make the code more intuitive by:
The changes affect example code in the documentation as well as actual implementation code, ensuring consistency across the entire project.