Skip to content

fix: add check for ADR to PR template#14

Merged
gemmadanks merged 7 commits intomainfrom
fix-add-adr-pr-check
Oct 15, 2025
Merged

fix: add check for ADR to PR template#14
gemmadanks merged 7 commits intomainfrom
fix-add-adr-pr-check

Conversation

@gemmadanks
Copy link
Copy Markdown
Owner

@gemmadanks gemmadanks commented Oct 15, 2025

What changes did you make (give an overview)?

  • Added item to PR template checklist for ADR
  • Removed unused sections of PR template

Prerequisites checklist

Please check each of the following before merging:

  • I have updated any relevant documentation
  • I have added unit tests for any new code
  • I have linked to any relevant issues
  • I will squash my commits and, where needed, edit the commit messages to make them more informative follow the conventional commits format
  • This pull request is ready to merge (leave unchecked to indicate a WIP)

What is the purpose of this pull request?

Check all that apply:

  • Documentation update
  • Bug fix
  • New feature
  • Changes to an existing feature
  • Refactor
  • Increasing test coverage
  • Other, please explain:

Copilot AI review requested due to automatic review settings October 15, 2025 17:50
Copy link
Copy Markdown
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

Adds an ADR (Architecture Decision Record) requirement to the pull request template and removes ancillary reviewer guidance sections.

  • Added checklist item to ensure ADRs are created for architecturally significant changes
  • Removed reviewer focus and miscellaneous comments sections

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

Comment thread .github/pull_request_template.md Outdated
Comment thread .github/pull_request_template.md Outdated
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 15, 2025 21:03
Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


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

Comment thread .github/pull_request_template.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 15, 2025 21:03
Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


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

Comment thread .github/pull_request_template.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 15, 2025 21:04
Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


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

@gemmadanks gemmadanks merged commit c15d43d into main Oct 15, 2025
25 checks passed
@gemmadanks gemmadanks deleted the fix-add-adr-pr-check branch October 15, 2025 21:21
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.

2 participants