Skip to content

fix: release process - #393

Merged
NicoleMGomes merged 1 commit into
mainfrom
fix/release
Oct 15, 2026
Merged

NicoleMGomes merged 1 commit into
mainfrom
fix/release

Conversation

@NicoleMGomes

@NicoleMGomes NicoleMGomes commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Disclaimer: Do not include SAP-internal or customer-specific information in this PR (e.g. internal system URLs, customer names, tenant IDs, or confidential configurations). This is a public repository.

Description

  • Fix bump version requirement on skill for code review
  • Fix label creation on skill for code review

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

How to Test

Describe how reviewers can test your changes:

  1. Open claude
  2. Run /review-pr

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages
  • (New modules only) I have attached e2e evidence showing the module working against a real BTP service
  • (Security-sensitive changes only) I have run /security-review and addressed all ❌ findings

@NicoleMGomes
NicoleMGomes marked this pull request as ready for review October 15, 2026 12:30
@NicoleMGomes
NicoleMGomes requested a review from a team as a code owner October 15, 2026 12:30
@NicoleMGomes
NicoleMGomes merged commit c4a5724 into main Oct 15, 2026
12 of 13 checks passed
@NicoleMGomes
NicoleMGomes deleted the fix/release branch October 15, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants