Skip to content

chore: add jira in-review transition for ready PRs - #584

Open
joaodordio wants to merge 1 commit into
mainfrom
chore/jira-in-review-on-ready-pr
Open

chore: add jira in-review transition for ready PRs#584
joaodordio wants to merge 1 commit into
mainfrom
chore/jira-in-review-on-ready-pr

Conversation

@joaodordio

@joaodordio joaodordio commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds a GitHub Action that transitions linked SDK-* Jira tickets to In Review when a non-draft PR is opened or marked ready for review.
  • Explicitly skips draft PRs so WIP stays in In Progress.
  • Uses repo secrets JIRA_BASE_URL, JIRA_USER_EMAIL, and JIRA_API_TOKEN (personal token for now; swap to a bot later).

No customer facing changes

Required follow-up

  1. Set the three secrets on this repo.
  2. Disable the native SDK Jira automation rule that transitions on Pull request created → In Review, otherwise drafts will still move.

Test plan

  • Set secrets on the repo
  • Open a draft PR with an SDK-* key → ticket stays In Progress
  • Mark that PR ready for review → ticket moves to In Review
  • Open a non-draft PR with an SDK-* key → ticket moves to In Review
  • Confirm PRs without an SDK-* key are no-ops

Skip draft PRs and only transition when a non-draft PR is opened or marked ready for review, so draft work stays In Progress.
@joaodordio
joaodordio requested a review from a team as a code owner August 7, 2026 23:51
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.43%. Comparing base (33e6f8e) to head (6c89c76).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #584   +/-   ##
=======================================
  Coverage   74.43%   74.43%           
=======================================
  Files          57       57           
  Lines        2742     2742           
  Branches      821      821           
=======================================
  Hits         2041     2041           
- Misses        660      700   +40     
+ Partials       41        1   -40     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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