Skip to content

Fix cli issue#1604

Merged
Krishanx92 merged 1 commit intowso2:mainfrom
Krishanx92:cli
Mar 31, 2026
Merged

Fix cli issue#1604
Krishanx92 merged 1 commit intowso2:mainfrom
Krishanx92:cli

Conversation

@Krishanx92
Copy link
Copy Markdown
Contributor

@Krishanx92 Krishanx92 commented Mar 31, 2026

Purpose

Explain why this feature or fix is required. Describe the underlying problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe what solutions this feature or fix introduces to address the problems outlined above.

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI. Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Automation tests

  • Unit tests

    Code coverage information

  • Integration tests

    Details about the test cases and coverage

Security checks

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Summary by CodeRabbit

  • Chores
    • Updated sample policies configuration by removing uppercase-body and count-letters policies
    • Added new transform-payload-case policy to sample policies

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 50148382-be52-446e-9881-0a1357ab4e35

📥 Commits

Reviewing files that changed from the base of the PR and between d2e1c6e and d1dd224.

📒 Files selected for processing (1)
  • gateway/sample-policies/build.yaml

Walkthrough

The build configuration file updates the sample policies list by removing two existing policies (uppercase-body and count-letters) and introducing a new policy (transform-payload-case).

Changes

Cohort / File(s) Summary
Build Configuration
gateway/sample-policies/build.yaml
Updated policies list: removed uppercase-body and count-letters policies, added transform-payload-case policy with corresponding file path reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Out go the old with their cases and counts,
In hops the new transform with its mount,
The payload case shifts left and right,
A config refresh, simple and light! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title 'Fix cli issue' is vague and generic, using non-descriptive terms that don't convey what the actual change is about. The PR actually updates policies in gateway/sample-policies/build.yaml, not a CLI issue. Update the title to clearly describe the main change, such as 'Replace uppercase-body and count-letters policies with transform-payload-case policy'.
Description check ⚠️ Warning The PR description is entirely unfilled—it only contains the template placeholders with no actual content in any required sections, making it impossible to understand the purpose, goals, or implementation details of the changes. Fill in all required sections: explain why the policy changes are needed, what problems they solve, how they're implemented, and any relevant documentation or test coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

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

@Krishanx92 Krishanx92 merged commit 25c6cc1 into wso2:main Mar 31, 2026
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants