Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change jsxImportSource to react #2184

Merged
merged 2 commits into from
Jul 14, 2023
Merged

Change jsxImportSource to react #2184

merged 2 commits into from
Jul 14, 2023

Conversation

connor-baer
Copy link
Member

@connor-baer connor-baer commented Jul 14, 2023

Fixes #2163.

Purpose

Emotion.js is still included in the bundle for stable components because it is configured as the jsxImportSource.

Approach and changes

  • Changed the jsxImportSource from @emotion/react to react and added a compiler directive override to all files that still use Emotion.js

Definition of done

  • Development completed
  • Reviewers assigned
  • Unit and integration tests
  • Meets minimum browser support
  • Meets accessibility requirements

@connor-baer connor-baer requested a review from a team as a code owner July 14, 2023 12:17
@connor-baer connor-baer requested review from tareqlol and removed request for a team July 14, 2023 12:17
@changeset-bot
Copy link

changeset-bot bot commented Jul 14, 2023

🦋 Changeset detected

Latest commit: d2e373a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sumup/circuit-ui Patch

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

@vercel
Copy link

vercel bot commented Jul 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
oss-circuit-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 2:39pm

@codecov
Copy link

codecov bot commented Jul 14, 2023

Codecov Report

Merging #2184 (d2e373a) into next (e1db849) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next    #2184      +/-   ##
==========================================
- Coverage   96.57%   96.56%   -0.02%     
==========================================
  Files         259      259              
  Lines       20481    20523      +42     
  Branches     1938     1928      -10     
==========================================
+ Hits        19780    19818      +38     
- Misses        693      702       +9     
+ Partials        8        3       -5     
Impacted Files Coverage Δ
...ges/circuit-ui/components/Calendar/RangePicker.tsx 86.20% <100.00%> (+0.49%) ⬆️
...dar/components/CalendarWrapper/CalendarWrapper.tsx 100.00% <100.00%> (ø)
...mponents/CalendarTagTwoStep/CalendarTagTwoStep.tsx 51.53% <100.00%> (-2.60%) ⬇️
packages/circuit-ui/components/Grid/Col/Col.ts 100.00% <100.00%> (ø)
packages/circuit-ui/components/Grid/Grid/Grid.ts 100.00% <100.00%> (ø)
packages/circuit-ui/components/Grid/Row/Row.ts 100.00% <100.00%> (ø)
packages/circuit-ui/components/Grid/utils.ts 100.00% <100.00%> (ø)
...it-ui/components/InlineElements/InlineElements.tsx 100.00% <100.00%> (ø)
packages/circuit-ui/components/Sidebar/Sidebar.tsx 100.00% <100.00%> (ø)
...nents/Sidebar/components/Aggregator/Aggregator.tsx 95.23% <100.00%> (-1.15%) ⬇️
... and 11 more

... and 36 files with indirect coverage changes

@connor-baer connor-baer merged commit 4b7dd4d into next Jul 14, 2023
7 of 8 checks passed
@connor-baer connor-baer deleted the fix/jsx-import-source branch July 14, 2023 15:13
@connor-baer connor-baer mentioned this pull request Aug 8, 2023
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.

2 participants