Skip to content

RELEASE: mssql-python-odbc 18.6.2.2 - #807

Merged
Jahnvi Thakkar (jahnvi480) merged 9 commits into
mainfrom
jahnvi/release-odbc-18-6-2-2
Sep 24, 2026
Merged

Jahnvi Thakkar (jahnvi480) merged 9 commits into
mainfrom
jahnvi/release-odbc-18-6-2-2

Conversation

@jahnvi480

@jahnvi480 Jahnvi Thakkar (jahnvi480) commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Work Item / Issue Reference

AB#48265


Summary

Prepare mssql-python-odbc 18.6.2.2 as a packaging-only release of the existing Microsoft ODBC Driver 18.6 binaries.

  • Bump the authoritative ODBC package version and update its packaging-revision documentation.
  • Release the merged libc-specific Linux wheel packaging: manylinux wheels contain glibc-based driver variants, and musllinux wheels contain only the Alpine/musl variant, for x86-64 and ARM64.
  • Keep the mssql-python ODBC dependency pin derived from the authoritative package version.

No native driver binaries or main mssql-python version are changed. This is a draft release-preparation PR; it does not publish packages. Rebuilt artifacts must be validated before the separate ODBC PyPI release.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 22, 2026 18:07
@github-actions github-actions Bot added the pr-size: small Minimal code update label Sep 22, 2026
@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

PR Performance Report

Performance assessment superseded.

The pull request revision changed before publication completed.

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.

Copilot review overview

🟡 Changes recommended

Update the macOS version parser before approval.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity

Open (1)
What changed in this PR

Packaging-only release of mssql-python-odbc 18.6.2.2 using existing ODBC binaries.

Changes:

  • Bumps the authoritative package version.
  • Documents the libc-specific wheel packaging revision and dependency pin.
  • Updates provider tests to validate the declared version.

A critical macOS build issue remains: the version parser must accept the fourth version component.

File Description
tests/​test_026_odbc_provider.py Validates the provider version dynamically.
README.md Documents the release and dependency updates.
mssql_python_odbc/​__init__.py Defines version 18.6.2.2.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread mssql_python_odbc/__init__.py
@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

📊 Code Coverage Report

🔥 Diff Coverage

100%


🎯 Overall Coverage

84%


📈 Total Lines Covered: 9054 out of 10697
📁 Project: mssql-python


Diff Coverage

Diff: main...HEAD, staged and unstaged changes

No lines with coverage information in this diff.


📋 Files Needing Attention

📉 Files with overall lowest coverage (click to expand)
mssql_python.pybind.performance_counter.hpp: 0.7%
mssql_python.pybind.logger_bridge.cpp: 57.9%
mssql_python.pybind.ddbc_bindings.h: 64.1%
mssql_python.pybind.logger_bridge.hpp: 70.8%
mssql_python.pybind.ddbc_bindings.cpp: 78.4%
mssql_python.pybind.connection.connection_pool.cpp: 82.3%
mssql_python.pybind.connection.connection.cpp: 82.5%
mssql_python.logging.py: 86.2%
mssql_python.pooling.py: 90.1%
mssql_python.pybind.fetch_temporal.hpp: 92.1%

🔗 Quick Links

⚙️ Build Summary 📋 Coverage Details

View Azure DevOps Build

Browse Full Coverage Report

@jahnvi480
Jahnvi Thakkar (jahnvi480) marked this pull request as ready for review September 24, 2026 05:31

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.

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified, and all reviewed changes are consistent.

Review effort: Lite
Findings: 1 High severity

Open (1)

Copilot AI review requested due to automatic review settings September 24, 2026 05:33

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.

Copilot review overview

🟢 Approval recommended

No unresolved blocking issues were identified.

Review effort: Lite
Findings: 1 High severity

Open (1)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

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.

Copilot review overview

🟢 Approval recommended

No unresolved blocking issues were identified.

Review effort: Lite
Findings: 1 High severity

Open (1)

Pin published audit wheels explicitly while retaining exact dependency, hash, and ownership validation. Keep recorded release provenance source-bound.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 24, 2026 06:01
@github-actions github-actions Bot added pr-size: medium Moderate update size and removed pr-size: small Minimal code update labels Sep 24, 2026

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.

Copilot review overview

🔵 Needs a closer look

The macOS dylib version parser and a Conda provenance baseline test require fixes.

Review effort: Lite
Findings: 1 High severity

Open (1)

Revert the audit-trigger and published-baseline changes. Handle conda-only trigger scoping in a separate main-based PR instead.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 24, 2026 06:07
@github-actions github-actions Bot added pr-size: small Minimal code update and removed pr-size: medium Moderate update size labels Sep 24, 2026

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.

Copilot review overview

🟢 Approval recommended

All reviewed changes are consistent with the packaging-only release.

Review effort: Lite
Findings: 1 High severity

Open (1)

Allow an optional fourth numeric version component while preserving invalid-version failures and configuring both bundled macOS architectures.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 24, 2026 06:24
@github-actions github-actions Bot added pr-size: medium Moderate update size and removed pr-size: small Minimal code update labels Sep 24, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added pr-size: small Minimal code update and removed pr-size: medium Moderate update size labels Sep 24, 2026

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.

Copilot review overview

🔵 Needs a closer look

Rebuilt release artifacts require validation before approval.

Review effort: Lite
Findings: None

Resolved since last review (1)

Copilot AI review requested due to automatic review settings September 24, 2026 06:27

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.

Copilot review overview

🟢 Approval recommended

The reviewed packaging, versioning, documentation, and test updates have no unresolved blocking issues.

Review effort: Lite
Findings: None

Copilot AI review requested due to automatic review settings September 24, 2026 07:22

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.

Copilot review overview

🟢 Approval recommended

All reviewed changes are consistent with the packaging-only release scope.

Review effort: Lite
Findings: None

@jahnvi480
Jahnvi Thakkar (jahnvi480) merged commit 1c14bb9 into main Sep 24, 2026
31 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-size: small Minimal code update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants