Skip to content

chore: update @cartridge/ui#2344

Merged
broody merged 1 commit intomainfrom
update-ui
Jan 21, 2026
Merged

chore: update @cartridge/ui#2344
broody merged 1 commit intomainfrom
update-ui

Conversation

@tarrencev
Copy link
Contributor

Updates @cartridge/ui dependency to point to commit 78d9aba

@vercel
Copy link

vercel bot commented Jan 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
controller-example-next Ready Ready Preview Jan 21, 2026 9:21pm
keychain Ready Ready Preview Jan 21, 2026 9:21pm
keychain-storybook Ready Ready Preview Jan 21, 2026 9:21pm

Request Review

@claude
Copy link
Contributor

claude bot commented Jan 21, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@codecov
Copy link

codecov bot commented Jan 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.01%. Comparing base (0e418ec) to head (27e04af).
⚠️ Report is 37 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2344      +/-   ##
==========================================
- Coverage   20.89%   20.01%   -0.88%     
==========================================
  Files         307      321      +14     
  Lines       31079    32666    +1587     
  Branches      880      884       +4     
==========================================
+ Hits         6493     6538      +45     
- Misses      24574    26116    +1542     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@broody broody merged commit 0b2eadf into main Jan 21, 2026
13 of 14 checks passed
@broody broody deleted the update-ui branch January 21, 2026 22:43
@claude
Copy link
Contributor

claude bot commented Jan 21, 2026

Claude finished @tarrencev's task —— View job


Analyzing controller repository changes for documentation updates

Task: Analyze PR #2344 changes and update documentation in docs-repo accordingly.

PR Summary:

  • Title: chore: update @cartridge/ui
  • Description: Updates @cartridge/ui dependency to commit 78d9aba
  • Files changed: package.json, pnpm-lock.yaml, pnpm-workspace.yaml

Todo List:

  • Read the changed files to understand the scope of the UI dependency update
  • Check the docs-repo directory to understand existing documentation structure
  • Analyze if the @cartridge/ui update has user-facing implications requiring documentation updates
  • Update documentation files if needed, or confirm no updates are required

Analysis Complete

The PR #2344 is a chore dependency update that:

  1. Updates @cartridge/ui dependency from one commit to another (78d9aba)
  2. No functional changes: The changes are purely in package management files (package.json, pnpm-lock.yaml, pnpm-workspace.yaml)
  3. No user-facing changes: This is an internal UI component library update with no API changes, new features, or breaking changes
  4. Documentation review: The only reference to @cartridge/ui in the docs is in the React example (/docs-repo/src/pages/controller/examples/react.md) where it imports Button component, but this import usage remains unchanged

Conclusion: No documentation updates are required. This is a standard dependency update that doesn't affect the public API, user-facing functionality, or integration patterns documented in the docs repository.

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