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

Release v13.12.0 #1875

Merged
merged 9 commits into from
Mar 4, 2024
Merged

Release v13.12.0 #1875

merged 9 commits into from
Mar 4, 2024

Conversation

booc0mtaco
Copy link
Contributor

@booc0mtaco booc0mtaco commented Mar 4, 2024

13.12.0 (2024-03-04)

Storybook

Features

Bug Fixes

booc0mtaco and others added 9 commits February 21, 2024 18:13
- cover remaining utilities in EDS _utils
fix: replace named lodash imports with deep ones for better tree shaking

Instead of

  import { debounce } from 'lodash';

use

  import debounce from 'lodash/debounce';

This helps with tree shaking. In particular I ran into issues with this
when trying to update Remix apps to use Vite.

An alternative would be to use 'lodash-es' instead, which is esm modules.
- model it to work and look like what exists for InputField
- add tests and new snapshots
- add rule override for disabled components
* add max height for Select.Options
* update test to snapshot when open
* add padding for visibility
* remove unnecessary waitFor call
Usages in our Remix apps will not need to deeply import anymore (in fact,
they cannot with Vite due to how the bundling works)

  - import type {ClickableStyleProps} from "@chanzuckerberg/eds/lib/components/ClickableStyle";
  + import type {ClickableStyleProps} from "@chanzuckerberg/eds";

We need the ClickableStyleProps for our custom Remix Link component, so
we properly integrate the Remix and EDS Link components.

Re-exporting the props is easier to work with. The deep import in edu-stack was always a hack.
* chore: test commit lint change (1/n)
* chore: test spaces (2/n)
* chore: test github.event.distinct_size (3/n)
* chore: test the original command's meaning (4/n)
* chore: filter on just commit IDs for counting (5/n)
* chore: tidy up pull request template (6/n)
* chore: document where the filtering technique comes from (7/n)
@booc0mtaco booc0mtaco requested a review from a team as a code owner March 4, 2024 15:19
Copy link

codecov bot commented Mar 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.62%. Comparing base (7bffc1a) to head (37ceba8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1875      +/-   ##
==========================================
+ Coverage   91.31%   92.62%   +1.31%     
==========================================
  Files         148      148              
  Lines        2855     2862       +7     
  Branches      776      780       +4     
==========================================
+ Hits         2607     2651      +44     
+ Misses        221      195      -26     
+ Partials       27       16      -11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

github-actions bot commented Mar 4, 2024

size-limit report 📦

Path Size
components 98.2 KB (+0.14% 🔺)
styles 39.99 KB (+0.15% 🔺)

@booc0mtaco
Copy link
Contributor Author

Thanks @ahuth ! doing this last small PR for the lingering v13 stuff, then will finish up work on v14 later today

@booc0mtaco booc0mtaco merged commit 492af63 into main Mar 4, 2024
14 checks passed
@booc0mtaco booc0mtaco deleted the release-v13.12.0 branch March 4, 2024 17:32
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.

3 participants