Skip to content

feat!: remove claude workflow templates#88

Merged
timovankissing-leanix merged 1 commit intomainfrom
devx-767-remove-claude-workflow-templates
Apr 23, 2026
Merged

feat!: remove claude workflow templates#88
timovankissing-leanix merged 1 commit intomainfrom
devx-767-remove-claude-workflow-templates

Conversation

@timovankissing-leanix
Copy link
Copy Markdown
Contributor

The shared claude workflows require an SAP AI proxy,
which is no longer available in CI. They are therefor
not functional and we do not want to encourage teams
to add them to their repo.

The shared claude workflows require an SAP AI proxy,
which is no longer available in CI. They are therefor
not functional and we do not want to encourage teams
to add them to their repo.
@internal-ai-adoption-analytics
Copy link
Copy Markdown

internal-ai-adoption-analytics Bot commented Apr 23, 2026

📊 Survey: Please help us track internal AI adoption. This is not for evaluation, only for metrics. You can update this anytime before merge.
🤖 Which AI developer tool(s), if any, were used to assist with this PR? Please tick one or more boxes below:

  • None
  • Cline
  • Claude Code
  • Cursor
  • GitHub Copilot
  • Other

🧑‍💻 If AI was used, what was it mainly used for?

  • Coding / Implementation
  • Testing (unit, integration, e2e)
  • Documentation
  • Research / Investigation
  • Refactoring / Cleanup
  • Configuration / DevOps
  • Other

Thank you in advance for your feedback.💙🙏🎉
Here's a joke to brighten your day: Tomorrow's headline: 'Local AI Finally Understands Sarcasm, World Not Ready' 📰

@hyperspace-insights
Copy link
Copy Markdown

Summary

The following content is AI-generated and provides a summary of the pull request:


Remove Claude Workflow Templates

Chore

🗑️ Removed the shared Claude workflow templates as they relied on an SAP AI proxy that is no longer available in CI, making them non-functional. Keeping these templates would risk encouraging teams to adopt them despite their broken state.

Changes

  • workflow-templates/claude-code-review.properties.json: Removed the properties file for the Claude code review workflow template.
  • workflow-templates/claude-code-review.yml: Removed the Claude code review workflow template.
  • workflow-templates/claude.properties.json: Removed the properties file for the general Claude workflow template.
  • workflow-templates/claude.yml: Removed the general Claude workflow template.

⚠️ Breaking Change: These workflow templates are no longer available. Teams that have already integrated them should remove or replace the affected workflows, as the required SAP AI proxy is no longer supported in CI.


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.20.23

  • Summary Prompt: Default Prompt
  • Output Template: Default Template
  • LLM: anthropic--claude-4.6-sonnet
  • File Content Strategy: Full file content
  • Correlation ID: eba294b0-9ada-4845-9c9d-c2bb90b1f21c
  • Event Trigger: pull_request.opened

Copy link
Copy Markdown

@hyperspace-insights hyperspace-insights Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description contains a minor typo ("therefor" instead of "therefore"), but that's a non-code item and per review rules, typos in PR descriptions are not actionable review comments (only in code/identifiers). The changes themselves are purely deletions with no remaining code to comment on. The removal is clean and complete — all four associated files (both .yml templates and their .properties.json metadata files) are removed together consistently.

The PR is straightforward and correct: four files are cleanly removed with no orphaned references in the repository. The change is well-scoped and the rationale (SAP AI proxy no longer available in CI) is clearly stated.

PR Bot Information

Version: 1.20.23

  • Event Trigger: pull_request.opened
  • Correlation ID: eba294b0-9ada-4845-9c9d-c2bb90b1f21c
  • File Content Strategy: Full file content
  • LLM: anthropic--claude-4.6-sonnet

@timovankissing-leanix timovankissing-leanix merged commit 68c747a into main Apr 23, 2026
3 checks passed
@timovankissing-leanix timovankissing-leanix deleted the devx-767-remove-claude-workflow-templates branch April 23, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants