-
Notifications
You must be signed in to change notification settings - Fork 13
fix: pin azure-ai-projects to <2.0.0 in tooling-extensions-azureaifoundry #233
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
Changes from all commits
4cadf89
042ce61
c3de9a9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -60,11 +60,12 @@ dev-dependencies = [ | |
| "langchain-openai", | ||
| ] | ||
|
|
||
| # Override semantic-kernel's azure-ai-projects constraint to allow 2.x | ||
| # semantic-kernel 1.39.3 constrains azure-ai-projects~=1.0.0b12 but this repo needs >=2.0.0b1 | ||
| # The functionality is compatible; only the version constraint was tightened | ||
| # azure-ai-projects is capped at <2.0.0 because: | ||
| # 1. semantic-kernel>=1.39.3 requires azure-ai-projects~=1.0.0b12 (<2.0.0) | ||
| # 2. tooling-extensions-azureaifoundry uses the 1.x Agents API (project_client.agents.update_agent) | ||
| # which was removed in azure-ai-projects 2.x | ||
| override-dependencies = [ | ||
| "azure-ai-projects >= 2.0.0b1", | ||
| "azure-ai-projects>=1.0.0b12,<2.0.0", | ||
| ] | ||
|
Comment on lines
+63
to
69
|
||
|
|
||
| # Centralized version constraints for all external dependencies | ||
|
|
@@ -80,7 +81,7 @@ constraint-dependencies = [ | |
|
|
||
| # --- Azure Services --- | ||
| "azure-ai-agents >= 1.0.0b251001", | ||
| "azure-ai-projects >= 2.0.0b1", | ||
| "azure-ai-projects>=1.0.0b12,<2.0.0", | ||
| "azure-identity >= 1.12.0", | ||
|
Comment on lines
82
to
85
|
||
| "azure-monitor-ingestion >= 1.0.0", | ||
| "azure-monitor-opentelemetry-exporter >= 1.0.0b39", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new requirement strings omit the spaces used by the other entries in this file (e.g.,
"azure-ai-agents >= ..."vs"azure-ai-projects>=..."). For consistency/readability, consider using the same spacing convention forazure-ai-projectsin bothoverride-dependenciesandconstraint-dependencies.