Skip to content

chore: upgrade @primer/react to v38 - #505

Open
Miablo wants to merge 1 commit into
github-community-projects:mainfrom
Miablo:deps/primer-react-v38
Open

Miablo wants to merge 1 commit into
github-community-projects:mainfrom
Miablo:deps/primer-react-v38

Conversation

@Miablo

@Miablo Miablo commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Pull Request

  • Upgrades @primer/react 36.27.0 → 38.39.0 and @primer/octicons-react to 19.35.0
  • Primer v38 removes the library's dependency on styled-components, which drops support for the sx prop and styled-system props — migrates every affected component off sx to CSS Modules
  • New .module.css files added alongside each migrated component, plus a shared src/app/styles/shared.module.css for common auth-card styling
  • No component behavior changes — this is a styling-mechanism migration only

Addresses the @primer/react v38 step of the dependency upgrade plan in #412.

Working UI post-changes

Screenshot 2026-09-12 at 9 08 29 PM Screenshot 2026-09-12 at 9 58 46 PM Screenshot 2026-09-12 at 9 59 06 PM Screenshot 2026-09-12 at 9 59 11 PM

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run npm run format and fix any formatting issues that have been introduced
  • run npm run lint and fix any linting issues that have been introduced
  • run npm run test and run tests

@Miablo Miablo self-assigned this Sep 13, 2026
@Miablo
Miablo marked this pull request as ready for review September 13, 2026 23:24
@Miablo
Miablo requested review from a team as code owners September 13, 2026 23:24
@Miablo
Miablo enabled auto-merge September 13, 2026 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant