update js #2681
Merged
update js #2681
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kernicPanel
force-pushed
the
kernicpanel/renovate/js-dependencies
branch
15 times, most recently
from
24556e7 to
9c2bb02
Compare
kernicPanel
force-pushed
the
kernicpanel/renovate/js-dependencies
branch
2 times, most recently
from
4727d79 to
fbb6070
Compare
Update webtorrent js dependencies.
kernicPanel
force-pushed
the
kernicpanel/renovate/js-dependencies
branch
from
fbb6070 to
117f5af
Compare
Update lambdas js dependencies.
kernicPanel
force-pushed
the
kernicpanel/renovate/js-dependencies
branch
3 times, most recently
from
d699c2c to
bb615a2
Compare
jbpenrath
reviewed
Jan 27, 2025
src/aws/lambda-convert/package.json
Outdated
| @@ -11,7 +11,7 @@ | |||
| "aws-sdk": "2.1692.0", | |||
| "clamscan": "2.4.0", | |||
| "he": "1.2.0", | |||
| "jimp": "1.6.0", | |||
| "jimp": "0.22.12", | |||
There was a problem hiding this comment.
When I pinned a dependency, I often use '<' to continue to install patches.
Suggested change
| "jimp": "0.22.12", | |
| "jimp": ">=0.22.12 <1", | |
src/frontend/package.json
Outdated
| @@ -75,7 +75,7 @@ | |||
| "react-router-dom": "6.22.2", | |||
| "styled-components": "5.3.11", | |||
| "typescript": "5.7.2", | |||
| "zustand": "4.5.2" | |||
| "zustand": "4.5.5" | |||
There was a problem hiding this comment.
Ditto
Suggested change
| "zustand": "4.5.5" | |
| "zustand": ">=4.5.5 <5" |
src/frontend/package.json
Outdated
| @@ -72,7 +71,7 @@ | |||
| "react": "18.2.0", | |||
| "react-dom": "18.2.0", | |||
| "react-intl": "7.1.0", | |||
| "react-router-dom": "6.22.2", | |||
| "react-router-dom": "6.28.2", | |||
There was a problem hiding this comment.
Ditto
Suggested change
| "react-router-dom": "6.28.2", | |
| "react-router-dom": ">=6.28.2 <7", |
| screen, | ||
| render as testingLibraryRender, | ||
| } from '@testing-library/react'; | ||
| import { Grommet, ResponsiveContext, ThemeType } from 'grommet'; | ||
| import MatchMediaMock from 'jest-matchmedia-mock'; | ||
| import { BreadCrumbsProvider, GlobalStyles, Nullable, theme } from 'lib-common'; | ||
| import React, { CSSProperties, ComponentProps, ReactElement } from 'react'; | ||
| import { act } from 'react-dom/test-utils'; |
There was a problem hiding this comment.
I remember this method is deprecated in favor of react/test-utils.
https://react.dev/warnings/react-dom-test-utils
But I don't understand why you cannot continue to use act from testing-library/react ?
jbpenrath
approved these changes
Jan 28, 2025
jimp v1 requires node 20 to run. We postpone this update for now.
Update frontend js dependencies.
Update sentry/browser dependency and code.
Remove useless fetch-mock types.
Something is broken with our import config and fetch-mock v10
Update typescript and code. Add @typescript-eslint/typescript-estree.
Update grommet and code.
Zustand v5 introduce breaking changes. We are updating to last v4.
kernicPanel
force-pushed
the
kernicpanel/renovate/js-dependencies
branch
from
0224bb0 to
b0ca449
Compare
Update react-router to last v6 and pin it. Remove @types/react-router.
Pin mermaid types to 9.1.0.
Update dompurify and code.
Pin eslint, @typescript-eslint/eslint-plugin, @typescript-eslint/parser
Pin p2p-media-loader-hlsjs.
Update testing-library and code.
Update react-test-renderer.
Update react to 18.3.1 and code.
Update react 19 and code.
Wait for grommet compatibility. grommet/grommet#7458
E2e tests needed some fix, due do one of recent dependency update.
kernicPanel
force-pushed
the
kernicpanel/renovate/js-dependencies
branch
from
13cbfa6 to
f3ee70b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
Update js dependencies.
#2662 is old and must be manually replaced.