Skip to content

Remove the old workflows from the package directories - #87

Merged
jwrosewell merged 1 commit into
rcfrom
chore/remove-package-workflows
Nov 13, 2026
Merged

jwrosewell merged 1 commit into
rcfrom
chore/remove-package-workflows

Conversation

@jwrosewell

@jwrosewell jwrosewell commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Closes #33.

Problem

Each package directory still holds the workflows of the repository it came from, 49 files under eight .github/workflows folders (device-detection/cloud, device-detection/onpremise, ip-intelligence, location, pipeline/cloudrequestengine, pipeline/core, pipeline/did and pipeline/engines). GitHub runs only the workflows at the root of a repository, so they never run here. On the mirrors that Packagist reads, mirror.yml puts each package directory at the root, so there they ran as live workflows, and all 21 on the mirrors were disabled by hand on 4 November 2026 after the UTM link lint failed there, as #33 says.

What changed

  • The eight package-level .github/workflows folders are deleted. Every package directory held only workflows under .github, so each .github folder goes with them. The root workflows are untouched.
  • The comment in .github/workflows/mirror.yml no longer says that every package keeps its old workflows. The mirror token still needs Workflows write, because the push that deletes them from the mirrors changes files under .github/workflows.

The next push of main runs the Branch job of mirror.yml, which splits each directory and pushes it to its mirror's main, so the deletions reach the mirrors then. No release tag is needed for that.

Releases

Nothing in this change is package code. Under #37 each package's Ignore list in ci/packages.json leaves out <path>/.github/workflows/**, so this change releases no package. Once both are merged those Ignore entries match nothing, and they can stay, so that a workflow added to a package directory by mistake still releases nothing.

Checks

CI only, so it merges on green quick checks and an independent review with no full build, under the 12 November decision for changes that touch only CI scripts and their notes.

The quick checks on head f593d21 passed, being the environment variable names check (run 43476788705), only rc into main (run 43476789224) and the UTM link lint (run 43476788664). A search of the branch finds no .github folder under any package directory, and the root .github/workflows is unchanged apart from the comment in mirror.yml.

Each package directory still held the workflows of its old repository.
They never run here, and on the mirrors they land at the repository root,
where they ran as live workflows until they were disabled by hand. The
next push of main deletes them from the mirrors too.
@jwrosewell
jwrosewell marked this pull request as ready for review November 13, 2026 23:06
@jwrosewell
jwrosewell merged commit 7d41f97 into rc Nov 13, 2026
3 checks passed
@jwrosewell
jwrosewell deleted the chore/remove-package-workflows branch November 13, 2026 23:06
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.

1 participant