Skip to content

Get actual resolved SDK versions instead of wildcards#210

Merged
abdulanu0 merged 6 commits into
mainfrom
users/anabdul/fix_sdk_versions
Feb 5, 2026
Merged

Get actual resolved SDK versions instead of wildcards#210
abdulanu0 merged 6 commits into
mainfrom
users/anabdul/fix_sdk_versions

Conversation

@abdulanu0

Copy link
Copy Markdown
Contributor

No description provided.

@abdulanu0 abdulanu0 requested a review from a team as a code owner February 3, 2026 21:53
Copilot AI review requested due to automatic review settings February 3, 2026 21:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modifies the version detection logic to retrieve actual resolved SDK versions from installed packages instead of relying on wildcard version specifications in manifest files.

Changes:

  • Updates Get-PythonVersions to use uv pip list to get installed versions before falling back to pyproject.toml parsing
  • Updates Get-NodejsVersions to read versions from installed node_modules packages before falling back to package.json
  • Updates Get-DotnetVersions to use dotnet list package with resolved versions before falling back to csproj file parsing

Comment thread scripts/Update-ReadmeVersions.ps1 Outdated
Comment thread scripts/Update-ReadmeVersions.ps1 Outdated
Comment thread scripts/Update-ReadmeVersions.ps1 Outdated
@github-actions

github-actions Bot commented Feb 3, 2026

Copy link
Copy Markdown

✅ All E2E Tests Passed

Sample Status Result
Python OpenAI success
Node.js OpenAI success
Node.js LangChain success
.NET Semantic Kernel success
.NET Agent Framework success

📦 SDK Versions: View the workflow summary for detailed SDK version information.

View full test details

pontemonti
pontemonti previously approved these changes Feb 3, 2026
@abdulanu0 abdulanu0 enabled auto-merge (squash) February 3, 2026 22:36
Comment thread README.md Outdated

@rahuldevikar761 rahuldevikar761 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see comment

- Remove static version text that required commits to update
- Add dynamic badges that fetch versions from npm, PyPI, and NuGet
- Delete update-readme-versions.yml workflow (no longer needed)
- Delete Update-ReadmeVersions.ps1 script (no longer needed)

Addresses feedback about needing dynamic versions that work on scheduled runs
without requiring README modifications.
Comment thread README.md Outdated
…mmaries

- Remove all hardcoded version tables from README
- Direct users to E2E workflow step summaries for actual resolved versions
- No README commits needed - versions are displayed dynamically in workflow logs
- E2E tests use flexible constraints (>=, ^, *-beta.*) to get latest packages

Addresses feedback about avoiding README modifications for version updates.
@abdulanu0 abdulanu0 merged commit 233c9fe into main Feb 5, 2026
36 checks passed
@abdulanu0 abdulanu0 deleted the users/anabdul/fix_sdk_versions branch February 5, 2026 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants