Skip to content

chore(skill): use imperative mood for CSS Modules changeset phrasing#1041

Merged
DreaminDani merged 1 commit into
mainfrom
chore/update-css-modules-changeset-phrasing
May 26, 2026
Merged

chore(skill): use imperative mood for CSS Modules changeset phrasing#1041
DreaminDani merged 1 commit into
mainfrom
chore/update-css-modules-changeset-phrasing

Conversation

@DreaminDani
Copy link
Copy Markdown
Contributor

@DreaminDani DreaminDani commented May 26, 2026

Summary

  • Update the CSS Modules migration skill so the templated changeset reads in imperative mood: Migrate <Name> from styled-components to css modules with no change in behavior.
  • Replaces the previous descriptive form (migration of <Name> from styled-components to css modules. no behavior change.) after reviewer feedback that changeset bodies should read like git commit messages (as if preceded by an implicit "this change will:").
  • Two spots updated: the template snippet and the checklist line that pins the exact wording.

Test plan

  • Future migration PRs (CardHorizontal, CardPrimary, IconButton, SplitButton) use the new phrasing.
  • No other behavior in the skill changes.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only edits to an internal migration skill; no product code or release behavior changes.

Overview
Updates the component-css-modules-migration skill so the required Changesets body uses imperative wording: Migrate <Name> from styled-components to css modules with no change in behavior, instead of the old descriptive line (migration of <Name>... no behavior change.).

The same sentence is applied in the changeset template and in the pre-merge checklist that pins exact wording. No migration tooling, components, or runtime behavior change—only documentation for future CSS Modules migration PRs.

Reviewed by Cursor Bugbot for commit bd5c42e. Bugbot is set up for automated code reviews on this repo. Configure here.

Switch the templated changeset body from descriptive ("migration of X
from styled-components to css modules. no behavior change.") to
imperative ("Migrate X from styled-components to css modules with no
change in behavior") so it reads naturally after an implicit "this
change will:" preamble, matching git commit message conventions.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 26, 2026

⚠️ No Changeset found

Latest commit: bd5c42e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@workflow-authentication-public
Copy link
Copy Markdown
Contributor

Storybook Preview Deployed

✅ Preview URL: https://click-mufruavnx-clickhouse.vercel.app

Built from commit: 2073e26bcd157d5e7a56e622053de658fdf8f8d8

@DreaminDani DreaminDani merged commit 14465bb into main May 26, 2026
13 checks passed
@DreaminDani DreaminDani deleted the chore/update-css-modules-changeset-phrasing branch May 26, 2026 14:33
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