Skip to content

fix(source-control): babysit gate does not hold a CLEAN head that is behind a non-strict base #5955

Description

@kyle-sexton

Problem

The references say behind is a hard stop, but compute_branch_freshness only runs the compare when mergeStateStatus is BLOCKED. A CLEAN head that is behind its base under a non-strict ruleset is therefore not held. A merge queue would make this moot for queue bases. Pre-existing.

Evidence

  • plugins/source-control/skills/babysit-prs/scripts/babysit_delta.py:233 (compute_branch_freshness).

Suggested fix

Run the compare for CLEAN heads too (or whenever the base is not queue-managed), and hold when behind.

Found while working on #5935.

No activity

Activity on this issue will appear here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    needs-triageNot yet classified. Floor until a type and one priority tier are set.

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions