Skip to content

Improve pre-push hook granularity and CI maintenance checks #3258

@Standing-Man

Description

@Standing-Man

Background

The current pre-merge CI already has component detection for language-specific jobs, but several checks still run broadly:

  • If the files being pushed do not include files relevant to a language/tool, that language/tool hook should not run.
  • Check PR Title does not rerun when a PR title is edited.
  • common checks in _common.yml always run, even when the changed files cannot affect that check.
  • scripts/ci/license-headers.sh still has a Docker fallback for addlicense.

We should tighten CI execution so PRs run the checks that are relevant to the changed files, while preserving dependency-driven coverage for core changes.

Goals

  1. Make pre-push hooks run only when the pushed changes include files relevant to that hook.
  2. Ensure PR title validation reruns after a PR title is edited.
  3. Remove Docker from the license-header check path and evaluate replacing addlicense with HawkEye.

Component

None

Proposed solution

No response

Alternatives considered

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions