Conversation
🦋 Changeset detectedLatest commit: 16d6cee The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Claude finished @punkbit's task in 1m 36s —— View job Code Review
✅ Looks great! Clean and correct removal. A quick check of the actual logo map (
The previous normalization PRs (#897, #900) established the non-hyphenated forms as canonical, which made these aliases effectively broken. Removing them is safe and correct — no regression risk. No changeset needed since no public API behavior changes (the aliases weren't working anyway). |
|
Claude finished @punkbit's task in 45s —— View job Code Review
✅ Looks great! Clean and correct removal. Verified against
These aliases were already broken (resolving to keys that don't exist in the logo map), so removing them is safe with zero regression risk. One minor note: the changeset is marked as |
|
Claude finished @punkbit's task in 20s —— View job Code Review
✅ Looks great! Clean and correct removal. The 7 removed aliases in The sole remaining alias ( No public API regression — the removed aliases were already non-functional. |
📚 Storybook Preview Deployed✅ Preview URL: https://click-16nv9zt6a-clickhouse.vercel.app Built from commit: |
Why?
Remove unnecessary aliases (deprecated due to asset name normalisation)
How?
Preview?
N/A