fix: use never for empty params #319
Annotations
4 errors and 2 warnings
|
packages/remix-routes/src/__tests__/build.test.ts#L21
Error: Snapshot `build v1 routes 1` mismatched
- Expected
+ Received
- "declare module \"remix-routes\" {
- type URLSearchParamsInit = string | string[][] | Record<string, string> | URLSearchParams;
- // symbol won't be a key of SearchParams
- type IsSearchParams<T> = symbol extends keyof T ? false : true;
-
- type ExportedQuery<T> = IsSearchParams<T> extends true ? T : URLSearchParamsInit;
-
-
- export interface Routes {
-
- \"/\": {
- params: never,
- query: ExportedQuery<import('../app/root').SearchParams>,
- };
-
- \"/:lang?/about\": {
- params: {
- lang?: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/($lang)/about').SearchParams>,
- };
-
- \"/admin\": {
- params: never,
- query: ExportedQuery<import('../app/routes/admin/index').SearchParams>,
- };
-
- \"/admin/episodes\": {
- params: never,
- query: ExportedQuery<import('../app/routes/admin/episodes/index').SearchParams>,
- };
-
- \"/admin/episodes/:id\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/admin/episodes/$id').SearchParams>,
- };
-
- \"/admin/episodes/:id/comments\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/admin/episodes/$id/comments/index').SearchParams>,
- };
-
- \"/admin/episodes/new\": {
- params: never,
- query: ExportedQuery<import('../app/routes/admin/episodes/new').SearchParams>,
- };
+ "declare module "remix-routes" {
+ type URLSearchParamsInit = string | string[][] | Record<string, string> | URLSearchParams;
+ // symbol won't be a key of SearchParams
+ type IsSearchParams<T> = symbol extends keyof T ? false : true;
+
+ type ExportedQuery<T> = IsSearchParams<T> extends true ? T : URLSearchParamsInit;
+
- \"/auth\": {
- params: never,
- query: ExportedQuery<import('../app/routes/auth/__auth').SearchParams>,
- };
-
- \"/auth/login\": {
- params: never,
- query: ExportedQuery<import('../app/routes/auth/__auth/login').SearchParams>,
- };
-
- \"/blog/rss.xml\": {
- params: never,
- query: ExportedQuery<import('../app/routes/blog.rss[.]xml').SearchParams>,
- };
-
- \"/chats/:season/:episode\": {
- params: {
- season: string | number;episode: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/chats.$season.$episode').SearchParams>,
- };
-
- \"/chats/:season/:episode/:slug\": {
- params: {
- season: string | number;episode: string | number;slug: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/chats.$season.$episode.$slug').SearchParams>,
- };
-
- \"/credits\": {
- params: never,
- query: ExportedQuery<import('../app/routes/credits').SearchParams>,
- };
-
- \"/home\": {
- params: never,
- query: ExportedQuery<import('../app/routes/(static)/home').SearchParams>,
- };
-
- \"/jokes\": {
- params: never,
- query: ExportedQuery<import('../app/routes/jokes/index').SearchParams>,
- };
-
- \"/jokes/:jokeId\": {
- params: {
- jokeId: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/jokes/$jokeId').SearchParams>,
- };
-
- \"/people/:personId\": {
- params: {
- personId: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/people/$personId').SearchParams>,
- };
-
- \"/people/:personId/:planId/remove-plan\": {
- params: {
- personId: string | number;planId: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/people/$personId/$planId/remove-plan').SearchParams>,
- };
-
- \"/posts\": {
- params: never,
- query: ExportedQuery<import('../app/routes/posts/index').SearchParams>,
- };
-
- \"/posts/:id\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/posts/$id').SearchParams>,
- };
-
- \"/posts/*\": {
- params: {
- \"*\": string | number;
- } ,
- query: ExportedQuery<import('../
|
|
packages/remix-routes/src/__tests__/build.test.ts#L32
Error: Snapshot `build v2 routes 1` mismatched
- Expected
+ Received
- "declare module \"remix-routes\" {
- type URLSearchParamsInit = string | string[][] | Record<string, string> | URLSearchParams;
- // symbol won't be a key of SearchParams
- type IsSearchParams<T> = symbol extends keyof T ? false : true;
+ "declare module "remix-routes" {
+ type URLSearchParamsInit = string | string[][] | Record<string, string> | URLSearchParams;
+ // symbol won't be a key of SearchParams
+ type IsSearchParams<T> = symbol extends keyof T ? false : true;
+
+ type ExportedQuery<T> = IsSearchParams<T> extends true ? T : URLSearchParamsInit;
+
- type ExportedQuery<T> = IsSearchParams<T> extends true ? T : URLSearchParamsInit;
-
-
- export interface Routes {
-
- \"/\": {
- params: never,
- query: ExportedQuery<import('../app/root').SearchParams>,
- };
-
- \"/:lang?/about\": {
- params: {
- lang?: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/($lang).about').SearchParams>,
- };
-
- \"/admin\": {
- params: never,
- query: ExportedQuery<import('../app/routes/admin._index').SearchParams>,
- };
-
- \"/admin/episodes\": {
- params: never,
- query: ExportedQuery<import('../app/routes/admin.episodes._index').SearchParams>,
- };
-
- \"/admin/episodes/:id\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/admin.episodes.$id._index').SearchParams>,
- };
-
- \"/admin/episodes/:id/comments\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/admin.episodes.$id.comments').SearchParams>,
- };
-
- \"/admin/episodes/new\": {
- params: never,
- query: ExportedQuery<import('../app/routes/admin.episodes.new').SearchParams>,
- };
-
- \"/api/:id.json\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/api.$id[.]json').SearchParams>,
- };
-
- \"/auth\": {
- params: never,
- query: ExportedQuery<import('../app/routes/auth._auth').SearchParams>,
- };
-
- \"/auth/login\": {
- params: never,
- query: ExportedQuery<import('../app/routes/auth._auth.login').SearchParams>,
- };
-
- \"/blog/rss.xml\": {
- params: never,
- query: ExportedQuery<import('../app/routes/blog.rss[.]xml').SearchParams>,
- };
-
- \"/chats/:season/:episode\": {
- params: {
- season: string | number;episode: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/chats_.$season.$episode').SearchParams>,
- };
-
- \"/chats/:season/:episode/:slug\": {
- params: {
- season: string | number;episode: string | number;slug: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/chats_.$season.$episode.$slug').SearchParams>,
- };
-
- \"/credits\": {
- params: never,
- query: ExportedQuery<import('../app/routes/credits').SearchParams>,
- };
-
- \"/home\": {
- params: never,
- query: ExportedQuery<import('../app/routes/(static).home').SearchParams>,
- };
-
- \"/jokes\": {
- params: never,
- query: ExportedQuery<import('../app/routes/jokes._index').SearchParams>,
- };
-
- \"/jokes/:jokeId\": {
- params: {
- jokeId: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/jokes.$jokeId').SearchParams>,
- };
-
- \"/people/:personId\": {
- params: {
- personId: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/people.$personId').SearchParams>,
- };
-
- \"/people/:personId/:planId/remove-plan\": {
- params: {
- personId: string | number;planId: string | number;
- } ,
- query: ExportedQuery<import('../app/routes/people.$personId.$planId.remove-plan').SearchParams>,
- };
-
- \"/posts\": {
- params: never,
- query: ExportedQuery<import('../app/routes/posts._index').SearchParams>,
- };
-
- \"/posts/:id\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery
|
|
packages/remix-routes/src/__tests__/build.test.ts#L43
Error: Snapshot `build customPaths routes 1` mismatched
- Expected
+ Received
- "declare module \"remix-routes\" {
- type URLSearchParamsInit = string | string[][] | Record<string, string> | URLSearchParams;
- // symbol won't be a key of SearchParams
- type IsSearchParams<T> = symbol extends keyof T ? false : true;
+ "declare module "remix-routes" {
+ type URLSearchParamsInit = string | string[][] | Record<string, string> | URLSearchParams;
+ // symbol won't be a key of SearchParams
+ type IsSearchParams<T> = symbol extends keyof T ? false : true;
+
+ type ExportedQuery<T> = IsSearchParams<T> extends true ? T : URLSearchParamsInit;
+
- type ExportedQuery<T> = IsSearchParams<T> extends true ? T : URLSearchParamsInit;
-
-
- export interface Routes {
-
- \"/\": {
- params: never,
- query: ExportedQuery<import('../../src/root').SearchParams>,
- };
-
- \"/:lang?/about\": {
- params: {
- lang?: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/($lang).about').SearchParams>,
- };
-
- \"/admin\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/admin._index').SearchParams>,
- };
-
- \"/admin/episodes\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/admin.episodes._index').SearchParams>,
- };
-
- \"/admin/episodes/:id\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/admin.episodes.$id._index').SearchParams>,
- };
-
- \"/admin/episodes/:id/comments\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/admin.episodes.$id.comments').SearchParams>,
- };
-
- \"/admin/episodes/new\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/admin.episodes.new').SearchParams>,
- };
-
- \"/auth\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/auth._auth').SearchParams>,
- };
-
- \"/auth/login\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/auth._auth.login').SearchParams>,
- };
-
- \"/blog/rss.xml\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/blog.rss[.]xml').SearchParams>,
- };
-
- \"/chats/:season/:episode\": {
- params: {
- season: string | number;episode: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/chats_.$season.$episode').SearchParams>,
- };
-
- \"/chats/:season/:episode/:slug\": {
- params: {
- season: string | number;episode: string | number;slug: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/chats_.$season.$episode.$slug').SearchParams>,
- };
-
- \"/credits\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/credits').SearchParams>,
- };
-
- \"/home\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/(static).home').SearchParams>,
- };
-
- \"/jokes\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/jokes._index').SearchParams>,
- };
-
- \"/jokes/:jokeId\": {
- params: {
- jokeId: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/jokes.$jokeId').SearchParams>,
- };
-
- \"/people/:personId\": {
- params: {
- personId: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/people.$personId').SearchParams>,
- };
-
- \"/people/:personId/:planId/remove-plan\": {
- params: {
- personId: string | number;planId: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/people.$personId.$planId.remove-plan').SearchParams>,
- };
-
- \"/posts\": {
- params: never,
- query: ExportedQuery<import('../../src/routes/posts._index').SearchParams>,
- };
-
- \"/posts/:id\": {
- params: {
- id: string | number;
- } ,
- query: ExportedQuery<import('../../src/routes/posts.$id').SearchParams>,
- };
-
- \"/posts/*\": {
- params: {
-
|
|
|
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, pnpm/[email protected], actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
This job failed
Loading