From da034d76bc578ac7dcbdc8a15685d193c847f770 Mon Sep 17 00:00:00 2001 From: Joao Dordio Date: Fri, 17 Jul 2026 14:08:45 +0100 Subject: [PATCH 1/2] chore: adopt shared SDK PR template and auto-request sdk-team reviews - Replace PR template with the shared SDK template - Add workflow to request review from @Iterable/sdk-team on new PRs --- .github/pull_request_template.md | 20 +++++++++++++++---- .github/workflows/assign-reviewers.yml | 27 ++++++++++++++++++++++++++ 2 files changed, 43 insertions(+), 4 deletions(-) create mode 100644 .github/workflows/assign-reviewers.yml diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index b2cc4804e..b7186b267 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,7 +1,19 @@ -## ๐Ÿ”น Jira Ticket(s) if any +# ๐Ÿ“ Summary +> _This should be a 150 characters summary of the changes on this PR_ -* [SDK-XXXX](https://iterable.atlassian.net/browse/SDK-XXXX) +###### ๐ŸŽŸ๏ธ Jira Ticket: [SDK-XXXX](https://iterable.atlassian.net/browse/SDK-XXXX) -## โœ๏ธ Description +## ๐Ÿ“– Description +> Please provide a description of what this pull request does. -> Please provide a brief description of what this pull request does. +## ๐Ÿงช How to test? +> How to test the changes added? + +## ๐Ÿ“น Loom recording if applicable +> If it helps the reviewer, add a short Loom going over the changes or showcasing the change in behavior. + +#### ๐Ÿž Github Issues solved +> If this solves any open GH Issues, please link them here. + +#### ๐Ÿ“š Docs PR if applicable +> Open a PR on the [Docs Repo](https://github.com/Iterable/iterable-docs) documenting the changes. diff --git a/.github/workflows/assign-reviewers.yml b/.github/workflows/assign-reviewers.yml new file mode 100644 index 000000000..4051e2563 --- /dev/null +++ b/.github/workflows/assign-reviewers.yml @@ -0,0 +1,27 @@ +name: Assign SDK Team Reviewers + +on: + pull_request: + types: [opened, ready_for_review] + +permissions: + pull-requests: write + +jobs: + assign-reviewers: + if: github.event.pull_request.draft == false + runs-on: ubuntu-latest + steps: + - uses: actions/github-script@v7 + with: + script: | + try { + await github.rest.pulls.requestReviewers({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.payload.pull_request.number, + team_reviewers: ['sdk-team'], + }); + } catch (error) { + core.warning(`Failed to request sdk-team review: ${error.message}`); + } From 570cb8f32ceae615d0c686092bf44ca918365ff6 Mon Sep 17 00:00:00 2001 From: Joao Dordio Date: Fri, 17 Jul 2026 14:19:37 +0100 Subject: [PATCH 2/2] chore: require sdk-team review via CODEOWNERS instead of workflow --- .github/CODEOWNERS | 4 ++++ .github/workflows/assign-reviewers.yml | 27 -------------------------- 2 files changed, 4 insertions(+), 27 deletions(-) create mode 100644 .github/CODEOWNERS delete mode 100644 .github/workflows/assign-reviewers.yml diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 000000000..cb9846c5a --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,4 @@ +# https://docs.github.com/articles/about-code-owners +# Require review from the SDK team on every PR. + +* @Iterable/sdk-team diff --git a/.github/workflows/assign-reviewers.yml b/.github/workflows/assign-reviewers.yml deleted file mode 100644 index 4051e2563..000000000 --- a/.github/workflows/assign-reviewers.yml +++ /dev/null @@ -1,27 +0,0 @@ -name: Assign SDK Team Reviewers - -on: - pull_request: - types: [opened, ready_for_review] - -permissions: - pull-requests: write - -jobs: - assign-reviewers: - if: github.event.pull_request.draft == false - runs-on: ubuntu-latest - steps: - - uses: actions/github-script@v7 - with: - script: | - try { - await github.rest.pulls.requestReviewers({ - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.payload.pull_request.number, - team_reviewers: ['sdk-team'], - }); - } catch (error) { - core.warning(`Failed to request sdk-team review: ${error.message}`); - }