Skip to content

Commit f6486ea

Browse files
hkc-8010hkc-8010
authored
Merge branch 'main' into fix/asset-event-lock-contention
2 parents a222fd9 + ea15bd2 commit f6486ea

399 files changed

Lines changed: 15694 additions & 11626 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.apache-steward-overrides/README.md‎

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
<!--
2+
Licensed to the Apache Software Foundation (ASF) under one
3+
or more contributor license agreements. See the NOTICE file
4+
distributed with this work for additional information
5+
regarding copyright ownership. The ASF licenses this file
6+
to you under the Apache License, Version 2.0 (the
7+
"License"); you may not use this file except in compliance
8+
with the License. You may obtain a copy of the License at
9+
10+
http://www.apache.org/licenses/LICENSE-2.0
11+
12+
Unless required by applicable law or agreed to in writing,
13+
software distributed under the License is distributed on an
14+
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
KIND, either express or implied. See the License for the
16+
specific language governing permissions and limitations
17+
under the License.
18+
-->
19+
20+
# apache-steward overrides
21+
22+
Agent-readable instructions that override specific steps or
23+
behaviours of apache-steward framework skills, scoped to
24+
this adopter repo. Each override file is named after the
25+
framework skill it modifies (e.g. `pr-management-triage.md`
26+
overrides the `pr-management-triage` skill).
27+
28+
The framework skills consult this directory at run-time
29+
before executing default behaviour. See
30+
[`docs/setup/agentic-overrides.md`](https://github.com/apache/airflow-steward/blob/main/docs/setup/agentic-overrides.md)
31+
in the framework for the full contract.
32+
33+
**Hard rule**: never modify the snapshot under
34+
`<repo-root>/.apache-steward/`. Local mods go here.
35+
Framework changes go via PR to `apache/airflow-steward`.

‎.apache-steward-overrides/pr-management-config.md‎

Lines changed: 77 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,77 @@
1+
<!--
2+
Licensed to the Apache Software Foundation (ASF) under one
3+
or more contributor license agreements. See the NOTICE file
4+
distributed with this work for additional information
5+
regarding copyright ownership. The ASF licenses this file
6+
to you under the Apache License, Version 2.0 (the
7+
"License"); you may not use this file except in compliance
8+
with the License. You may obtain a copy of the License at
9+
10+
http://www.apache.org/licenses/LICENSE-2.0
11+
12+
Unless required by applicable law or agreed to in writing,
13+
software distributed under the License is distributed on an
14+
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
KIND, either express or implied. See the License for the
16+
specific language governing permissions and limitations
17+
under the License.
18+
-->
19+
20+
<!-- START doctoc generated TOC please keep comment here to allow auto update -->
21+
<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
22+
**Table of Contents** *generated with [DocToc](https://github.com/thlorenz/doctoc)*
23+
24+
- [Apache Airflow — pr-management-triage configuration](#apache-airflow--pr-management-triage-configuration)
25+
- [Identifiers](#identifiers)
26+
- [Project-specific labels](#project-specific-labels)
27+
- [Grace windows](#grace-windows)
28+
29+
<!-- END doctoc generated TOC please keep comment here to allow auto update -->
30+
31+
<!-- SPDX-License-Identifier: Apache-2.0
32+
https://www.apache.org/licenses/LICENSE-2.0 -->
33+
34+
# Apache Airflow — pr-management-triage configuration
35+
36+
This file is the **per-project configuration** for the
37+
[`pr-management-triage`](../../.claude/skills/pr-management-triage/SKILL.md) skill.
38+
It contains the concrete values for the Apache Airflow project.
39+
New adopters should copy this file into their own
40+
`<project-config>/pr-management-config.md` and replace every
41+
Airflow-specific value with their project's equivalents.
42+
43+
## Identifiers
44+
45+
| Key | Value | Used by |
46+
|---|---|---|
47+
| `committers_team` | `apache/airflow-committers` | `classify-and-act.md` row F5b — team-mention detection. Used to recognise PR comments that `@`-mention the project's committers as a maintainer-to-maintainer ping. |
48+
| `area_label_prefix` | `area:` | `classify-and-act.md`, `pr-management-stats` — area-label grouping. |
49+
50+
## Project-specific labels
51+
52+
Labels the skill applies or watches for. Each row maps a generic
53+
**framework concept** to whatever label string the adopter uses.
54+
If the project doesn't have a given concept, leave the value blank
55+
and the skill will skip that row of decision-table actions.
56+
57+
| Concept | Label | Notes |
58+
|---|---|---|
59+
| `ready_for_maintainer_review` | `ready for maintainer review` | Applied by the `mark-ready` action; used by `pr-management-code-review` as a default selector. |
60+
| `quality_violations_close` | `closed because of multiple quality violations` | Applied when a PR is closed for failing the project's PR quality criteria after multiple opportunities to fix. |
61+
| `suspicious_changes` | `suspicious changes detected` | Applied to first-time-contributor workflow approvals where the diff looks suspicious (binary blobs, unrelated CI changes, etc.). |
62+
| `work_in_progress` | | Airflow does not use a dedicated WIP label; the skill relies on draft status instead. |
63+
64+
## Grace windows
65+
66+
Tunable thresholds. These values were calibrated for Airflow's
67+
contributor traffic (~50–100 open PRs, triage sweep every 1–2
68+
days).
69+
70+
| Concept | Default | Project value |
71+
|---|---|---|
72+
| Stale-draft close threshold (triaged) | 7 days | 7 days |
73+
| Stale-draft close threshold (untriaged) | 14 days | 14 days |
74+
| Inactive-open → draft threshold | 28 days | 28 days |
75+
| Stale-review-ping cooldown | 7 days | 7 days |
76+
| Stale-workflow-approval threshold | 28 days | 28 days |
77+
| Stale-Copilot-review threshold | 7 days | 7 days |

‎.apache-steward-overrides/pr-management-triage-ci-check-map.md‎

Lines changed: 79 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,79 @@
1+
<!--
2+
Licensed to the Apache Software Foundation (ASF) under one
3+
or more contributor license agreements. See the NOTICE file
4+
distributed with this work for additional information
5+
regarding copyright ownership. The ASF licenses this file
6+
to you under the Apache License, Version 2.0 (the
7+
"License"); you may not use this file except in compliance
8+
with the License. You may obtain a copy of the License at
9+
10+
http://www.apache.org/licenses/LICENSE-2.0
11+
12+
Unless required by applicable law or agreed to in writing,
13+
software distributed under the License is distributed on an
14+
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
KIND, either express or implied. See the License for the
16+
specific language governing permissions and limitations
17+
under the License.
18+
-->
19+
20+
<!-- START doctoc generated TOC please keep comment here to allow auto update -->
21+
<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
22+
**Table of Contents** *generated with [DocToc](https://github.com/thlorenz/doctoc)*
23+
24+
- [Apache Airflow — pr-management-triage CI-check to doc-URL map](#apache-airflow--pr-management-triage-ci-check-to-doc-url-map)
25+
- [Table](#table)
26+
- [Notes](#notes)
27+
28+
<!-- END doctoc generated TOC please keep comment here to allow auto update -->
29+
30+
<!-- SPDX-License-Identifier: Apache-2.0
31+
https://www.apache.org/licenses/LICENSE-2.0 -->
32+
33+
# Apache Airflow — pr-management-triage CI-check to doc-URL map
34+
35+
This file is the **CI-check categorisation table** for the
36+
[`pr-management-triage`](../../.claude/skills/pr-management-triage/SKILL.md) skill's
37+
violations comments. It contains the concrete mapping for the
38+
Apache Airflow project. New adopters should copy this file into
39+
their own `<project-config>/pr-management-triage-ci-check-map.md`
40+
and replace every Airflow-specific pattern and URL with their
41+
project's equivalents.
42+
43+
## Table
44+
45+
Each row maps a **GitHub check name pattern** (case-insensitive
46+
substring match) to a **human-readable category name** the skill
47+
prints in the violations comment, plus a **doc URL** the skill
48+
links to.
49+
50+
| Pattern | Category | Doc URL |
51+
|---|---|---|
52+
| `static checks`, `pre-commit`, `prek` | Pre-commit / static checks | `https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst` |
53+
| `ruff` | Ruff (linting / formatting) | `https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst` |
54+
| `mypy-` | mypy (type checking) | `https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst` |
55+
| `unit test`, `test-` | Unit tests | `https://github.com/apache/airflow/blob/main/contributing-docs/09_testing.rst` |
56+
| `docs`, `spellcheck-docs`, `build-docs` | Build docs | `https://github.com/apache/airflow/blob/main/contributing-docs/11_documentation_building.rst` |
57+
| `helm` | Helm tests | `https://github.com/apache/airflow/blob/main/contributing-docs/testing/helm_unit_tests.rst` |
58+
| `k8s`, `kubernetes` | Kubernetes tests | `https://github.com/apache/airflow/blob/main/contributing-docs/testing/k8s_tests.rst` |
59+
| `build ci image`, `build prod image`, `ci-image`, `prod-image` | Image build | `https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst` |
60+
| `provider` | Provider tests | `https://github.com/apache/airflow/blob/main/contributing-docs/12_provider_distributions.rst` |
61+
| `*` (catch-all) | Other failing CI checks | `https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst` |
62+
63+
## Notes
64+
65+
- **Order matters.** The skill matches first-found; more-specific
66+
patterns are listed above broader ones (e.g. `mypy-airflow-core`
67+
matches the `mypy-` row).
68+
- **Mergeability fallback.** If the PR has `mergeable ==
69+
CONFLICTING`, the skill emits a separate "Merge conflicts"
70+
category linking to the project's git/rebase doc:
71+
72+
| Concept | Doc URL |
73+
|---|---|
74+
| Merge conflicts (rebase guide) | `https://github.com/apache/airflow/blob/main/contributing-docs/10_working_with_git.rst` |
75+
76+
- **Failing-CI fallback.** If `checks_state == FAILURE` but no
77+
failed check names are extractable, the skill emits a generic
78+
"Failing CI checks" entry pointing at the same doc URL as the
79+
catch-all row above.

0 commit comments

Comments
 (0)