Skip to content

ci: enable automatic label sync triggers - #106

Merged
Wahbeh-Mohammad merged 1 commit into
mainfrom
labels-enable-triggers
Sep 9, 2026
Merged

Wahbeh-Mohammad merged 1 commit into
mainfrom
labels-enable-triggers

Conversation

@Wahbeh-Mohammad

Copy link
Copy Markdown
Contributor

Step 6 of onboarding, the last one. Uncomments the push and schedule
triggers left off in #105.

Why the cron matters

It is the only thing that re-heals a label edited in the web UI. Between
dispatches, a hand-edited color or description just stays edited, and the YAML
in dexpace/.github quietly becomes advisory rather than authoritative.
Pruning is what makes the files the source of truth; the cron is what makes
pruning happen without anyone remembering.

Merging this runs a real pruning sync

Worth stating plainly: the push trigger matches this exact path, and
LABELS_PRUNE=true is set on this repo, so the merge commit triggers a sync
with pruning live — not a dry run.

That is safe because the repo has already converged, verified after the
manual sequence:

Labels 43 = the 42 in common,docs,package,sdk, plus javascript
adapters, v1/MVP already deleted
Migrated assignments all intact, checked as exact issue-number sets via REST
Milestone #1 v1/MVP 46 items

So the expected result of this merge is a run that creates nothing, updates
nothing and deletes nothing. If it does anything at all, something drifted
between now and the merge, and the run log will say what.

Step 6 of the onboarding order in dexpace/.github labels/README.md. The first
real sync has been read and applied, so the automatic triggers can come on.

The Monday cron is the part that matters. Between dispatches a label edited in
the web UI stays edited; the cron is what re-heals it. Without it the YAML in
dexpace/.github is advisory rather than authoritative.

Merging this fires a pruning sync immediately: the push trigger matches this
path, and LABELS_PRUNE is set on this repo. That is a no-op today because the
repo already converged — 42 labels present, adapters and v1/MVP already gone,
javascript excluded — so the merge doubles as a reconciliation check.
@Wahbeh-Mohammad
Wahbeh-Mohammad merged commit e8c165e into main Sep 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants