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

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

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant