Skip to content

build: add pull request template - #254

Merged
kacperzolkiewski merged 4 commits into
software-mansion:mainfrom
kacperzolkiewski:add-pull-request-template
Nov 17, 2025
Merged

kacperzolkiewski merged 4 commits into
software-mansion:mainfrom
kacperzolkiewski:add-pull-request-template

Conversation

@kacperzolkiewski

@kacperzolkiewski kacperzolkiewski commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

The template looks like this:

Summary

Explain the motivation for making this change: here are some points to help you:

  • What issues does the pull request solve? Please tag them so that they will get automatically closed once the PR is merged
  • What is the feature? (if applicable)
  • How did you implement the solution?
  • What areas of the library does it impact?

Test Plan

Provide clear steps so another contributor can reproduce the behavior or verify the feature works.
For example:

  • Steps to reproduce the bug (if this is a bug fix)
  • Steps to verify the new feature
  • Expected vs. actual results
  • Any special conditions or edge cases to test

Screenshots / Videos

Include any visual proof that helps reviewers understand the change — UI updates, bug reproduction or the result of the fix.

Compatibility

OS Implemented
iOS ✅❌
Android ✅❌

Comment thread .github/PULL_REQUEST_TEMPLATE.md Outdated
Comment thread .github/PULL_REQUEST_TEMPLATE.md Outdated
@exploIF
exploIF requested a review from szydlovsky November 17, 2025 10:14
kacperzolkiewski and others added 2 commits November 17, 2025 11:16
Co-authored-by: Igor Furgała <74370735+exploIF@users.noreply.github.com>
Co-authored-by: Igor Furgała <74370735+exploIF@users.noreply.github.com>

@szydlovsky szydlovsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good idea!

@kacperzolkiewski
kacperzolkiewski merged commit 835e78e into software-mansion:main Nov 17, 2025
5 checks passed
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.

3 participants