php / php/php-src

Ignore generated arginfo for the ABI break label

Open
#18,708 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

ABI break Category: Build System Feature Status: Needs Triage
Dominant language
C
Stars
40.4k
Forks
8.1k
Avg merge
2d 13h
Merged PRs (30d)
96

Description

Description

Any objections? From the initial #15682

ABI breaks are not supposed to happen after feature freeze, i.e. when
the PHP API numbers have been bumped. To make it easier to notice
inadvertent ABI breaks, we automatically add an "ABI break" label to
all PRs which modify public (aka. installed) header files.

And I would consider arginfo to qualify as non-public header files

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the referenced issue #15682 and locating the automation that adds the "ABI break" label to pull requests modifying installed headers. Determine how generated arginfo is identified, then verify that arginfo-only changes no longer receive the label; the payload names no file or test to run.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.