Repository navigation
Pinning Teams SDK version - #573
Rodrigo Brandão (rodrigobr-msft) wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The exact == dependency pin in install_requires is likely to cause downstream resolver conflicts and is inconsistent with the repository’s general use of bounded ranges for external dependencies.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This pull request updates the Teams hosting integration package’s dependency constraints to avoid unplanned upgrades of the microsoft-teams-api library.
Changes:
- Pin
microsoft-teams-apito2.0.16inlibraries/microsoft-agents-hosting-msteams/setup.pyinstead of allowing any2.xrelease.
File summaries
| File | Description |
|---|---|
| libraries/microsoft-agents-hosting-msteams/setup.py | Tightens the microsoft-teams-api dependency specification for the MSTeams hosting package. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
🟡 Changes recommended
The updated dependency specifier does not actually “pin” to 2.0.16 as stated in the PR description, so it won’t ensure consistent builds.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
This pull request makes a minor update to the
libraries/microsoft-agents-hosting-msteams/setup.pyfile, narrowing the version range for themicrosoft-teams-apidependency to ensure compatibility with versions >=2.0.16 but less than 2.1.