Meta: use `clsx` lib by Copilot · Pull Request #9664 · refined-github/refined-github · GitHub
Skip to content

Meta: use clsx lib#9664

Merged
fregante merged 9 commits into
mainfrom
copilot/use-classnames-as-cx
Jun 3, 2026
Merged

Meta: use clsx lib#9664
fregante merged 9 commits into
mainfrom
copilot/use-classnames-as-cx

Conversation

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor
  • Part of Refactor ideas #9496
  • Replaces classnames with clsx across all source files
  • Enforces import cx from 'clsx' via no-restricted-imports (bans import {clsx} from 'clsx') and a byo/clsx-import-as-cx selector rule (bans any default import not named cx)
  • Minor cx() call improvements (argument ordering, class grouping, readability)

Test URLs

Screenshot

Copilot AI changed the title feat: use classnames (cx) for dynamic className JSX attributes feat: use classnames (cx) for dynamic className JSX attributes Jun 1, 2026
Copilot AI requested a review from fregante June 1, 2026 09:47
@fregante fregante added the meta Related to Refined GitHub itself label Jun 1, 2026
@github-actions github-actions Bot changed the title feat: use classnames (cx) for dynamic className JSX attributes Use classnames (cx) for dynamic className JSX attributes Jun 1, 2026
@fregante fregante changed the title Use classnames (cx) for dynamic className JSX attributes Meta: use classnames lib Jun 1, 2026
@kidonng

kidonng commented Jun 1, 2026

Copy link
Copy Markdown
Member

@fregante fregante left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot also replace classnames with clsx

Comment thread eslint-rules/restricted-syntax.js Outdated
Comment thread source/features/conversation-activity-filter.tsx Outdated
Comment thread source/features/repo-avatars.tsx Outdated
Comment thread source/features/tag-changes-link.tsx Outdated

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Copilot AI changed the title Meta: use classnames lib Meta: use clsx lib Jun 1, 2026
Copilot AI requested a review from fregante June 1, 2026 14:25

This comment was marked as resolved.

@fregante fregante marked this pull request as ready for review June 1, 2026 16:57
Comment thread eslint.config.js Outdated
Copilot AI requested a review from fregante June 3, 2026 16:35
@fregante fregante enabled auto-merge (squash) June 3, 2026 17:04
@fregante fregante merged commit 1306ce9 into main Jun 3, 2026
17 checks passed
@fregante fregante deleted the copilot/use-classnames-as-cx branch June 7, 2026 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Related to Refined GitHub itself

Development

Successfully merging this pull request may close these issues.

3 participants