Add ai-training OWNERS team and reserve experimental/air#5605
Merged
Conversation
Adds the ai-training team alias and assigns it ownership of the new experimental/air and acceptance/experimental/air directories. Co-authored-by: Isaac
pardis-beikzadeh-db
approved these changes
Jun 15, 2026
pietern
approved these changes
Jun 15, 2026
Collaborator
Integration test reportCommit: 0b1bc4e
24 interesting tests: 15 SKIP, 7 KNOWN, 2 flaky
Top 20 slowest tests (at least 2 minutes):
|
Collaborator
Integration test reportCommit: 93e5e66
426 interesting tests: 344 MISS, 40 FAIL, 35 RECOVERED, 3 PANIC, 2 SKIP, 1 KNOWN, 1 flaky
Top 50 slowest tests (at least 2 minutes):
|
2 tasks
artchen-db
pushed a commit
to artchen-db/cli
that referenced
this pull request
Jun 18, 2026
## Changes Add `.agent/skills/onboard-team-area/SKILL.md`, a small skill that encodes the repeatable steps for onboarding a new team or reserving a new code area: 1. Add the team to `.github/OWNERTEAMS` (explicit `@members`, the source of truth). 2. Reserve `experimental/<area>/` and `acceptance/experimental/<area>/` with `.gitkeep`. 3. Map both paths to the team in `.github/OWNERS`. 4. Validate and open the PR. It also has a short experimental-vs-stable note and a pointer to the aitools graduation (PR databricks#4917) so a new team knows where code lives and how it later moves to a stable top-level command. ## Why PR databricks#5605 onboarded the ai-training team by hand-editing `OWNERTEAMS` + `OWNERS` and reserving the experimental dirs, but that recipe was not written down. This skill captures it so the next onboarding is one prompt. It lives in-repo under `.agent/skills` (next to `pr-checklist` and `bump-cli-compat`, not in an external plugin) so it versions alongside the OWNERS files it describes. ## Tests - [x] `./task checks` - [x] Frontmatter and structure match the existing `pr-checklist` and `bump-cli-compat` skills This pull request and its description were written by Isaac.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
A new team (AI training) is starting experimental work in the CLI. They need a reserved place for that work and clear ownership, so any PR touching it routes review to the right people through the maintainer-approval workflow.
Changes
Before: there was no
ai-trainingteam and noexperimental/airarea.Now:
team:ai-trainingalias to.github/OWNERTEAMSwith its members..gitkeepplaceholders:experimental/air/andacceptance/experimental/air/.team:ai-trainingin.github/OWNERSso the team owns them.The team alias lists members explicitly because it is the source of truth for maintainer approval (the workflow cannot resolve GitHub org team membership via the API).
Note: the OWNERTEAMS header comment lists a GitHub team-page URL for each existing team. I left
ai-trainingwithout one for now since the GitHub team may not exist yet; easy to add once it does.Test plan
node --test .github/scripts/owners.test.js .github/workflows/maintainer-approval.test.js(54 pass)./task checkspasses (tidy, whitespace, links, deadcode)This pull request and its description were written by Isaac.