From ac1b24df9bed354f8e36fecbf8d14611ebc5a239 Mon Sep 17 00:00:00 2001 From: Joao Dordio Date: Fri, 17 Jul 2026 14:08:49 +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 23fc776b..b7186b26 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_ -* [MOB-XXXX](https://iterable.atlassian.net/browse/MOB-XXXX) +###### ๐ŸŽŸ๏ธ Jira Ticket: [SDK-XXXX](https://iterable.atlassian.net/browse/SDK-XXXX) -## Description +## ๐Ÿ“– Description +> Please provide a description of what this pull request does. -## Test Steps \ No newline at end of file +## ๐Ÿงช 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 00000000..4051e256 --- /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 716dbc555d0dfe21744333a276d8f76c7da9e337 Mon Sep 17 00:00:00 2001 From: Joao Dordio Date: Fri, 17 Jul 2026 14:19:41 +0100 Subject: [PATCH 2/2] chore: require sdk-team review via CODEOWNERS instead of workflow --- .github/CODEOWNERS | 5 ++++- .github/workflows/assign-reviewers.yml | 27 -------------------------- 2 files changed, 4 insertions(+), 28 deletions(-) delete mode 100644 .github/workflows/assign-reviewers.yml diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 71d0dcc4..cb9846c5 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1 +1,4 @@ -* @mprew97 +# 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 4051e256..00000000 --- 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}`); - }