Skip to content

Conversation

bewithgaurav
Copy link
Collaborator

@bewithgaurav bewithgaurav commented Oct 3, 2025

Work Item / Issue Reference

ADO Work Item: Closed AB#39110

GitHub Issue: Closes #276


Summary

This pull request updates the pull request template to clarify how references to work items and issues should be formatted. The changes help ensure that references are properly closed when the pull request is merged.

Improvements to reference formatting:

  • Changed the ADO Work Item reference to use Closed AB#<WORK_ITEM_ID> instead of AB#<WORK_ITEM_ID>.
  • Changed the GitHub Issue reference to use Closes GitHub Issue: #<ISSUE_NUMBER> instead of GitHub Issue: #<ISSUE_NUMBER>.

@Copilot Copilot AI review requested due to automatic review settings October 3, 2025 07:00
Copy link
Contributor

@Copilot 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

Updates the PR template to include auto-closing keywords for both ADO work items and GitHub issues, enabling automatic closure when PRs are merged.

Key Changes

  • Added "Closed" keyword before ADO work item references
  • Added "Closes" keyword before GitHub issue references

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the pr-size: small Minimal code update label Oct 3, 2025
Copy link

github-actions bot commented Oct 3, 2025

📊 Code Coverage Report

🔥 Diff Coverage

100%


🎯 Overall Coverage

74%


📈 Total Lines Covered: 4181 out of 5612
📁 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.connection.connection.cpp: 67.6%
mssql_python.ddbc_bindings.py: 68.5%
mssql_python.pybind.ddbc_bindings.cpp: 69.3%
mssql_python.pybind.connection.connection_pool.cpp: 78.9%
mssql_python.cursor.py: 79.7%
mssql_python.connection.py: 81.7%
mssql_python.helpers.py: 84.7%
mssql_python.auth.py: 85.3%
mssql_python.type.py: 86.8%
mssql_python.pooling.py: 87.5%

🔗 Quick Links

⚙️ Build Summary 📋 Coverage Details

View Azure DevOps Build

Browse Full Coverage Report

jahnvi480
jahnvi480 previously approved these changes Oct 6, 2025
@bewithgaurav
Copy link
Collaborator Author

bewithgaurav commented Oct 8, 2025

Instead of closing, we should target updating the Issue with a comment saying the PR fix has been merged and this will be released in the next release cycle. And then close the issue.

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.

TEST: Update PR Template to auto-close tasks

2 participants