Skip to content

fix(twake-mui): Accept twake-icons 3 in the devDependencies - #221

Merged
Eriikah merged 1 commit into
mainfrom
fix/twake-icons-3-range
Oct 30, 2026
Merged

Eriikah merged 1 commit into
mainfrom
fix/twake-icons-3-range

Conversation

@Eriikah

@Eriikah Eriikah commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

The release of `@linagora/twake-icons` 3.0.0 broke `npm ci` on main (run): twake-mui's devDependency `^2.10.0` no longer matched the workspace, so npm wanted 2.17.0 from the registry, which is missing from the lock file.

  • Bump the devDependency to `^3.0.0` and refresh `package-lock.json`.
  • The peerDependency `>=2.10.0` stays: 3.x satisfies it.
  • twake-mui uses none of the icons renamed in 3.0.0.

Checked locally with `npm ci` and `npm run build`.

🤖 Generated with Claude Code

twake-icons went to 3.0.0 and ^2.10.0 no longer matched the workspace,
so npm ci looked for 2.17.0 from the registry and failed on the lock file.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5c0aae99-20c5-4f2a-9e06-cc67bbb6e16d
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@Eriikah
Eriikah merged commit 0369887 into main Oct 30, 2026
5 of 6 checks passed
@Eriikah
Eriikah deleted the fix/twake-icons-3-range branch October 30, 2026 15:18
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.

1 participant