Repository navigation
chore(ci): remove gh aw workflows - #1789
Conversation
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Remove workflow automation .github/workflows/agentics-maintenance-shopware-docs.yml, .github/workflows/pr-docs-check.md |
The maintenance and documentation-check workflows are removed. |
Remove agent guidance and setup .github/agents/agentic-workflows.md, .github/skills/agentic-workflows/SKILL.md, .github/workflows/copilot-setup-steps.yml |
The agent definition, skill router, and Copilot setup workflow are removed. |
Remove related repository configuration .github/aw/actions-lock.json, .github/chainguard/swcli.sts.yaml, .github/dependabot.yml, .rumdl.toml |
The action lock and OIDC configuration are removed. Dependabot no longer excludes the agentic workflow and lock workflow paths. The rumdl exclusions for the deleted agent and skill files are removed. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Other
Possibly related PRs
- shopware/shopware-cli#1773: Adds Dependabot exclusions for the workflow paths that this pull request removes.
- shopware/shopware-cli#1772: Updates the action lock and generated workflows that this pull request deletes.
- shopware/shopware-cli#1766: Adds rumdl exclusions for the agent and skill files that this pull request deletes.
Suggested reviewers: maltejanz, tturkowski
Merge Risk: 
8c237
-
The retired automation and its supporting configuration have no identified surviving dependency. The change is ready to merge.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly and concisely states that the GitHub Agentic Workflows are being removed, which matches the main change. Description check The description includes all required sections and explains the changes, rationale, testing, and related issue status. It is complete and relevant. Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.
-
Finishing Touches
-
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1789 +/- ##
=======================================
Coverage 68.49% 68.50%
=======================================
Files 499 499
Lines 33365 33365
=======================================
+ Hits 22855 22856 +1
+ Misses 10510 10509 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
LGTM 👍
But maybe we should shortly align / discuss in the team how we want to deal with docs going forward, to still ensure our docs stay up to date, and write that down somewhere. E.g. a decision record like "we tried GH AW for automatic docs update but didn't liked it because X, Y, ... . Instead we now do Z" 🤔 , could also be an ADR 🤷.
That way we would still have our reasoning in 6 or 12 month if ever the question comes back why we didn't automate that with GH AW.
*Or at least add a bit more background on the "why" section of this PR for our reasons to get rid of it 😅
Pascal Zarrad (pascal-zarrad)
left a comment
Pascal Zarrad (pascal-zarrad)
left a comment
There was a problem hiding this comment.
LGTM 👍
What changed?
Removed gh aw workflows and their supporting files and configuration exclusions.
Why?
Retire GitHub Agentic Workflows from this repository.
How was this tested?
mise run lint-workflowsandgit diff --checkpass. No gh aw references remain.Related issue or discussion
None.
Summary by CodeRabbit
Changes
Maintenance