search

aviator-co / av

A command line tool to manage stacked PRs with Aviator
https://aviator.co
MIT License
149 stars 12 forks source link

Do not push a branch if an ancestor doesn't have a PR #351

Closed draftcode closed 2 months ago

draftcode commented 2 months ago

https://linear.app/mergequeue/issue/MER-4307/swap-the-order-of-push-pr-update-to-avoid-the-unintended-ci-triggers

aviator-app[bot] commented 2 months ago

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes. Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.

Stack

  1. 👉 Do not push a branch if an ancestor doesn't have a PR #351 👈 (this pr)
See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.
aviator-app[bot] commented 2 months ago

FlexReview Summary

Based on the code complexity and the author's expertise score, these are the suggested reviewers:

See the list of alternate reviewers in the detailed breakdown below.

Detailed Breakdown Author’s expertise score for the modified files: - `internal/gh/ghui/push.go` (1.00) | Files | Reviewers | | --- | --- | | `internal/gh/ghui/push.go` † | **`@jainankit`** (score: 0.48, current review load: 3) | † Indicates that the file doesn't need an expert review. [(?)](https://docs.aviator.co/flexreview/concepts/approval-requirements) See full breakdown of the reviewers on the Aviator webapp.
jainankit commented 2 months ago

What's the rationale behind this change?