-
Notifications
You must be signed in to change notification settings - Fork 917
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
[Internal][Executor] Extract some common constants to promptflow._constants #2303
Conversation
SDK CLI Global Config Test Result devs/peiwen/extract_common_constants2 tests 2 ✅ 49s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
promptflow-tracing unit test result devs/peiwen/extract_common_constants 4 files 4 suites 21s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
promptflow-tracing e2e test result devs/peiwen/extract_common_constants 4 files 4 suites 1m 44s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
promptflow SDK CLI Azure E2E Test Result devs/peiwen/extract_common_constants 4 files 4 suites 3m 46s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
Executor Unit Test Result devs/peiwen/extract_common_constants754 tests 754 ✅ 3m 51s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
Executor E2E Test Result devs/peiwen/extract_common_constants210 tests 208 ✅ 7m 43s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
SDK CLI Test Result devs/peiwen/extract_common_constants 12 files 12 suites 40m 26s ⏱️ Results for commit 942e4b2. ♻️ This comment has been updated with latest results. |
Description
Extract some constants used by both executor server and sdk/cli to
promptflow._constants
.All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines