Skip to content

Conversation

@gargsaumya
Copy link
Contributor

Work Item / Issue Reference

AB#<WORK_ITEM_ID>

GitHub Issue: #<ISSUE_NUMBER>


Summary

This pull request updates the documentation to reflect the transition of the mssql-python driver to General Availability (GA), clarifies platform and authentication support, and reorganizes feature and roadmap information for clarity and accuracy. The most important changes are grouped below.

Documentation Updates:

  • The driver is now marked as Generally Available (GA) and ready for production use, replacing the previous "Public Preview" notice in README.md.
  • The support note for SUSE Linux ARM64 has been clarified to indicate that only x64 architecture is supported for SUSE deployments.

Authentication and Platform Support:

  • Entra ID ActiveDirectoryIntegrated authentication is now documented as supported on Windows, macOS, and Linux, with a note about required Kerberos/SSPI configuration.

Feature and Example Reorganization:

  • The DBAPI v2.0 compliance and Enhanced Pythonic Features sections have been moved and improved for clarity, and the example code now demonstrates a realistic query using Entra ID Interactive authentication.

Roadmap Updates:

  • The ROADMAP.md file has been replaced with a concise feature matrix table, showing planned and in-progress features with estimated timelines, instead of a narrative roadmap.

Copilot AI review requested due to automatic review settings November 18, 2025 06:37
@github-actions github-actions bot added the pr-size: medium Moderate update size label Nov 18, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request transitions the mssql-python driver from Public Preview to General Availability (GA), updates platform and authentication support documentation, and restructures the roadmap from narrative format to a concise feature matrix.

Key Changes:

  • Marks the driver as GA and production-ready, replacing the previous preview warning
  • Converts ROADMAP.md from narrative documentation to a structured feature matrix with timelines
  • Updates authentication documentation to reflect cross-platform support for ActiveDirectoryIntegrated and enhances the getting started example with realistic SQL queries using Entra ID authentication

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
ROADMAP.md Replaced narrative roadmap with feature matrix table showing planned features, statuses, and target timelines
README.md Updated GA status, clarified SUSE platform support, expanded ActiveDirectoryIntegrated authentication support to all platforms, reorganized documentation sections, and enhanced code example with realistic query

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

github-actions bot commented Nov 18, 2025

📊 Code Coverage Report

🔥 Diff Coverage

100%


🎯 Overall Coverage

77%


📈 Total Lines Covered: 5065 out of 6536
📁 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.logger_bridge.cpp: 59.2%
mssql_python.helpers.py: 66.6%
mssql_python.row.py: 67.4%
mssql_python.pybind.ddbc_bindings.cpp: 70.4%
mssql_python.pybind.connection.connection.cpp: 76.3%
mssql_python.ddbc_bindings.py: 79.6%
mssql_python.pybind.connection.connection_pool.cpp: 79.6%
mssql_python.pybind.ddbc_bindings.h: 79.7%
mssql_python.connection.py: 82.5%
mssql_python.cursor.py: 83.6%

🔗 Quick Links

⚙️ Build Summary 📋 Coverage Details

View Azure DevOps Build

Browse Full Coverage Report

Copy link
Collaborator

@bewithgaurav bewithgaurav left a comment

Choose a reason for hiding this comment

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

Had a chat with Sumit, lgtm

@gargsaumya gargsaumya merged commit e185841 into main Nov 18, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-size: medium Moderate update size

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants