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

Update some deps #1509

Merged
merged 2 commits into from
Feb 25, 2023
Merged

Update some deps #1509

merged 2 commits into from
Feb 25, 2023

Conversation

ahuth
Copy link
Contributor

@ahuth ahuth commented Feb 24, 2023

This PR updates some deps.

@ahuth ahuth requested a review from a team February 24, 2023 19:30
Comment on lines +24 to +25
// Don't run axe on this story. The video seems to cause asset preloading to fail (even when
// the video has preload="none").
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@github-actions
Copy link

github-actions bot commented Feb 24, 2023

size-limit report 📦

Path Size
components 118.07 KB (0%)
styles 3.1 KB (0%)

@codecov
Copy link

codecov bot commented Feb 24, 2023

Codecov Report

Merging #1509 (9a80e7f) into next (9d3e22e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next    #1509   +/-   ##
=======================================
  Coverage   91.91%   91.91%           
=======================================
  Files         277      277           
  Lines        4180     4180           
  Branches      755      755           
=======================================
  Hits         3842     3842           
  Misses        313      313           
  Partials       25       25           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -127,22 +127,22 @@
"@types/react-beautiful-dnd": "^13.1.3",
"@types/react-dom": "^18.0.11",
"@types/react-portal": "^4.0.4",
"axe-core": "4.4.3",
"chromatic": "^6.17.0",
"axe-core": "4.6.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

^4.6.3?
Iirc we kept it at 4.4.3 due to timeouts but it looks like we're just skipping that. Can we remove axe-core from the ignore in the renovate config?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add the caret back (shoulda been more clear lol sorry)

Copy link
Contributor

@jinlee93 jinlee93 left a comment

Choose a reason for hiding this comment

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

Sweet thanks!

@ahuth ahuth merged commit af97207 into next Feb 25, 2023
@ahuth ahuth deleted the ah-updates branch February 25, 2023 01:10
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