-
Notifications
You must be signed in to change notification settings - Fork 901
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
Refactor: remove unnecessary code from flow_executor.py #3641
Conversation
9c3c7a3
to
d853756
Compare
Hi, thank you for your interest in helping to improve the prompt flow experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. |
@microsoft/promptflow-sdk |
SDK CLI Global Config Test Result remove-unnecessary-code0 tests 0 ✅ 0s ⏱️ Results for commit 8a4f9f6. |
Executor E2E Test Result remove-unnecessary-code0 tests 0 ✅ 0s ⏱️ Results for commit 8a4f9f6. |
Executor Unit Test Result remove-unnecessary-code0 tests 0 ✅ 0s ⏱️ Results for commit 8a4f9f6. |
Hi, thank you for your interest in helping to improve the prompt flow experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. |
Hi, thank you for your contribution. Since there has not been recent engagement, we are going to close this out. Feel free to reopen if you'd like to continue working on these changes. Please be sure to remove the |
Description
I have found that they are not used in the current code in flow_executor.py.
Perhaps they were forgotten to be removed in past changes.
All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines