Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

CHORE: Update PR Template to auto-close tasks #275

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
bewithgaurav wants to merge 4 commits into main
base: main
Choose a base branch
Loading
from bewithgaurav/update_pr_template

Conversation

@bewithgaurav
Copy link
Collaborator

@bewithgaurav bewithgaurav commented Oct 3, 2025
edited
Loading

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 AI review requested due to automatic review settings October 3, 2025 07:00
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

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
edited
Loading

📊 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

Copy link
Collaborator Author

bewithgaurav commented Oct 8, 2025
edited
Loading

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

Reviewers

Copilot code review Copilot Copilot left review comments

@jahnvi480 jahnvi480 jahnvi480 left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

pr-size: small Minimal code update

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

TEST: Update PR Template to auto-close tasks

AltStyle によって変換されたページ (->オリジナル) /