Skip to content
This repository was archived by the owner on Jul 10, 2026. It is now read-only.

ci: remove obsolete label sync workflow#378

Merged
devantler merged 1 commit into
mainfrom
claude/remove-obsolete-label-sync
Jul 10, 2026
Merged

ci: remove obsolete label sync workflow#378
devantler merged 1 commit into
mainfrom
claude/remove-obsolete-label-sync

Conversation

@devantler

@devantler devantler commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

🤖 Generated by the Daily AI Assistant

Why

The scheduled Sync Labels workflow is obsolete and now fails on main because it fetches the retired devantler-tech/actions label file path. Label ownership has moved to the organization-level declarative label manifests, so keeping this workflow creates recurring noise instead of maintaining labels.

What

  • Remove .github/workflows/.sync-labels.yaml.
  • Update AGENTS.md so reusable-workflows maintenance no longer references the removed internal label-sync workflow.

Validation

  • git diff --check
  • yamllint .github/workflows/
  • Ruby stdlib YAML parse over .github/workflows/*.yaml

Notes

actionlint .github/workflows/*.yaml was attempted, but it is blocked by pre-existing unrelated code-quality permission-scope findings in ci.yaml, run-dotnet-tests.yaml, and validate-go-project.yaml.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Removed the scheduled GitHub Actions label-synchronization workflow and updated AGENTS.md to clarify that dot-prefixed maintenance workflows are excluded from the reusable workflow_call inventory.

Possibly related issues

  • devantler-tech/reusable-workflows#350: Directly tracks removal of the failing nightly Sync Labels workflow.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately summarizes the main change: removing the obsolete label sync workflow.
Description check ✅ Passed The description clearly matches the changeset by explaining the workflow removal and the AGENTS.md update.

Comment @coderabbitai help to get the list of available commands.

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Daily AI Assistant

@coderabbitai review

The PR description has been filled in with the workflow-removal rationale, scope, validation evidence, and the existing actionlint caveat so the pre-merge Description check can re-evaluate current PR metadata.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

@devantler Thanks for updating the PR description with the removal rationale, scope, validation evidence, and the actionlint caveat. I'll review the current state of the PR now.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@devantler

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

🤖 Generated by the Daily AI Assistant

The PR body currently contains the workflow-removal rationale, scope, validation evidence, and actionlint caveat, but the latest CodeRabbit pre-merge summary visible through GitHub is still the older missing-description result from before that body update. Please refresh the pre-merge checks against the current PR metadata.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

@devantler Understood — I'll re-run the review now so the pre-merge checks pick up the current PR description and metadata.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@devantler devantler marked this pull request as ready for review July 10, 2026 06:17
@devantler devantler merged commit 99a3a60 into main Jul 10, 2026
53 checks passed
@devantler devantler deleted the claude/remove-obsolete-label-sync branch July 10, 2026 06:29
@github-project-automation github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board Jul 10, 2026
@botantler-1

botantler-1 Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.6.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@botantler-1 botantler-1 Bot added the released label Jul 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant