Skip to content

Add changeset 🦋#511

Merged
brionmario merged 1 commit intowso2:mainfrom
brionmario:fix-tables
Mar 19, 2026
Merged

Add changeset 🦋#511
brionmario merged 1 commit intowso2:mainfrom
brionmario:fix-tables

Conversation

@brionmario
Copy link
Copy Markdown
Member

@brionmario brionmario commented Mar 19, 2026

Purpose

This pull request introduces a patch update to both @wso2/oxygen-ui-icons-react and @wso2/oxygen-ui. The key improvements are enhanced flexibility for the ListingTableCellIcon component and the addition of a new icon.

Component flexibility and icon additions:

  • Updated ListingTableCellIcon to accept ReactNode, allowing for more flexible icon or content rendering.
  • Added a new Lambda icon to the icon set.

Related Issues

  • N/A

Related PRs

  • N/A

Checklist

  • Followed the CONTRIBUTING guidelines.
  • Manual test round performed and verified.
  • Documentation provided. (Add links if there are any)
  • Unit tests provided. (Add links if there are any)
  • Storybook stories updated/added (if applicable)

Security checks

Summary by CodeRabbit

Release Notes

  • New Features
    • Added a new Lambda icon to the icon library
    • Updated ListingTableCellIcon component to support ReactNode

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 19, 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: 70b96fbf-01a9-4a2f-84b4-bf363af338e2

📥 Commits

Reviewing files that changed from the base of the PR and between 41470ed and b5dde05.

📒 Files selected for processing (1)
  • .changeset/tall-carrots-ring.md

Walkthrough

A new Changeset metadata file documents patch releases for two packages: @wso2/oxygen-ui-icons-react and @wso2/oxygen-ui. The updates include allowing ReactNode in ListingTableCellIcon and adding a new Lambda icon.

Changes

Cohort / File(s) Summary
Release Metadata
.changeset/tall-carrots-ring.md
Added changeset file marking patch releases for icon and UI packages with updates to ListingTableCellIcon flexibility and new icon addition.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A carrot tall, a ring so fine,
New icons shine in grand design,
Lambda glows with React's delight,
Changes documented, all is bright!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Add changeset 🦋' is vague and generic, describing only the mechanical action of adding a changeset file without conveying the actual technical changes or improvements being introduced. Revise the title to be more specific about the actual changes, such as 'Allow ReactNode in ListingTableCellIcon and add Lambda icon' or 'Enhance ListingTableCellIcon flexibility and add Lambda icon'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description is mostly complete, covering the Purpose section with clear details about the changes and maintaining the template structure with all required sections present.
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
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@brionmario brionmario changed the title Fix tables Add changeset 🦋 Mar 19, 2026
@brionmario brionmario merged commit 4421663 into wso2:main Mar 19, 2026
1 check 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