Skip to content

chore: update PR template and auto-assign sdk-team reviewers#69

Open
joaodordio wants to merge 2 commits into
mainfrom
chore/update-pr-template-and-reviewers
Open

chore: update PR template and auto-assign sdk-team reviewers#69
joaodordio wants to merge 2 commits into
mainfrom
chore/update-pr-template-and-reviewers

Conversation

@joaodordio

Copy link
Copy Markdown
Member

Summary

  • Adopt the shared SDK PR template (Summary, Jira, Description, How to test, Loom, GH Issues, Docs PR).
  • Add .github/workflows/assign-reviewers.yml to request review from @Iterable/sdk-team on new (non-draft) PRs, placing the team in the Reviewers section.
  • Remove .github/CODEOWNERS (@Iterable/mobile) so only sdk-team is requested, matching the other SDK repos.

Test plan

  • Open a test PR against this branch and confirm the new template renders.
  • Confirm sdk-team is auto-requested as a reviewer once merged to the default branch.

- Replace PR template with the shared SDK template
- Add workflow to request review from @Iterable/sdk-team on new PRs
- Remove CODEOWNERS in favor of the reviewer workflow
@joaodordio
joaodordio requested a review from a team as a code owner July 17, 2026 13:10
@qltysh

qltysh Bot commented Jul 17, 2026

Copy link
Copy Markdown

Qlty


⚠️ Comments skipped @joaodordio doesn't have a Qlty seat in Iterable.

Qlty doesn't post analysis or coverage comments for contributors without a seat. An authorized user can grant @joaodordio a seat from this pull request's page in Qlty.

@joaodordio
joaodordio requested a review from a team July 17, 2026 13:26

@jferrao-itrbl jferrao-itrbl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jferrao-itrbl jferrao-itrbl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jferrao-itrbl jferrao-itrbl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@sumeruchat sumeruchat 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.

LGTM

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