-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
deps: Add missing peer dependencies #16751
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
base: develop
Are you sure you want to change the base?
Changes from all commits
c182072
0327fac
58ba2d6
0ec83a7
5288be7
e7a3875
21c1379
89badf6
8e4e86b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,9 +27,9 @@ | |
"dependencies": { | ||
"@aws-sdk/client-s3": "^3.552.0", | ||
"@hapi/hapi": "^21.3.10", | ||
"@nestjs/common": "11.0.16", | ||
"@nestjs/core": "10.4.6", | ||
"@nestjs/platform-express": "10.4.6", | ||
"@nestjs/common": "11.1.3", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. bumping & aligning to latest as that fixes some transitive dependency security issue. |
||
"@nestjs/core": "11.1.3", | ||
"@nestjs/platform-express": "11.1.3", | ||
"@sentry/aws-serverless": "9.33.0", | ||
"@sentry/core": "9.33.0", | ||
"@sentry/node": "9.33.0", | ||
|
@@ -74,7 +74,10 @@ | |
"@types/amqplib": "^0.10.5", | ||
"@types/node-cron": "^3.0.11", | ||
"@types/node-schedule": "^2.1.7", | ||
"globby": "11" | ||
"file-type": "^20.4.1", | ||
"globby": "11", | ||
"react": "^18.3.1", | ||
"zod": "^3.24.1" | ||
}, | ||
"config": { | ||
"mongodbMemoryServer": { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,7 +30,7 @@ | |
"postpack": "ember ts:clean" | ||
}, | ||
"dependencies": { | ||
"@babel/core": "^7.24.4", | ||
"@babel/core": "^7.27.7", | ||
"@embroider/macros": "^1.16.0", | ||
"@sentry/browser": "9.33.0", | ||
"@sentry/core": "9.33.0", | ||
|
@@ -71,7 +71,7 @@ | |
"ember-source": "~4.12.4", | ||
"ember-template-lint": "~4.16.1", | ||
"eslint-plugin-ember": "11.9.0", | ||
"eslint-plugin-n": "16.0.1", | ||
"eslint-plugin-n": "15.0.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. downgrading because this actually requires eslint 8 |
||
"eslint-plugin-qunit": "8.0.0", | ||
"loader.js": "~4.7.0", | ||
"qunit": "~2.22.0", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
import type { ClientOptions, Context } from '@sentry/core'; | ||
import { captureException, getClient, getTraceMetaTags, logger } from '@sentry/core'; | ||
import type { VueOptions } from '@sentry/vue/src/types'; | ||
import type { CapturedErrorContext } from 'nitropack'; | ||
import type { CapturedErrorContext } from 'nitropack/types'; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this apparently changed in a newer nitropack version 😭 |
||
import type { NuxtRenderHTMLContext } from 'nuxt/app'; | ||
import type { ComponentPublicInstance } from 'vue'; | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,7 +6,7 @@ export type RollupConfig = { | |
plugins: unknown[]; | ||
}; | ||
|
||
export type SolidStartInlineConfig = Parameters<typeof defineConfig>[0]; | ||
export type SolidStartInlineConfig = NonNullable<Parameters<typeof defineConfig>[0]>; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. type is now possible undefined, which leads to problems. |
||
|
||
export type SolidStartInlineServerConfig = { | ||
hooks?: { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this (subject) is not defined for this test, apparently some changed webpack version emits or does not emit the empty file anymore, but this should be OK.