Skip to content

Fix PR template to use proper GitHub issue linking syntax#1701

Merged
tamirkamara merged 3 commits into
mainfrom
copilot/fix-1700
Aug 18, 2025
Merged

Fix PR template to use proper GitHub issue linking syntax#1701
tamirkamara merged 3 commits into
mainfrom
copilot/fix-1700

Conversation

Copilot AI commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

The current PR template uses "This PR fixes issue #XX" which doesn't trigger GitHub's automatic issue linking and closing functionality. This prevents PRs from being automatically associated with issues in the development tab and stops issues from being closed when PRs are merged.

GitHub requires specific keywords directly followed by the issue number to enable automatic linking:

  • Fixes #123
  • Closes #123
  • Resolves #123
  • This PR fixes issue #123 (current format)
  • Fixes issue #123

Updated the PR template to use the standard Fixes #XX format, which will:

  1. Automatically link the PR to the referenced issue in GitHub's development tab
  2. Automatically close the issue when the PR is merged

This is a minimal change that removes unnecessary words while maintaining clarity and enabling proper GitHub integration.

Fixes #1700.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>
Copilot AI changed the title [WIP] PR Template doesn't automatically close the mentioned issue Fix PR template to use proper GitHub issue linking syntax Aug 18, 2025
Copilot AI requested a review from tamirkamara August 18, 2025 07:07
@tamirkamara tamirkamara marked this pull request as ready for review August 18, 2025 07:07
@tamirkamara

Copy link
Copy Markdown
Contributor

@copilot your description in this PR doesn't adhere to the format specified in the PR template for this repo.

Copilot AI commented Aug 18, 2025

Copy link
Copy Markdown
Contributor Author

@copilot your description in this PR doesn't adhere to the format specified in the PR template for this repo.

You're absolutely right. I've updated the PR description to follow the proper template format with the required sections: "Change Description", "Issue reference", and "Checklist". The description now adheres to the format specified in the PR template.

@SharonHart

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@tamirkamara tamirkamara merged commit a1695ba into main Aug 18, 2025
32 checks passed
@tamirkamara tamirkamara deleted the copilot/fix-1700 branch August 29, 2025 20:41
prokopidis pushed a commit to prokopidis/presidio that referenced this pull request Jun 23, 2026
…1701)

* Initial plan

* Fix PR template to use proper GitHub issue linking syntax

Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>
Co-authored-by: Sharon Hart <sharonh.dev@gmail.com>
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.

PR Template doesn't automatically close the mentioned issue

3 participants