Skip to content

Commit

Permalink
fix(nextjs): cleanup old options for server executor (proxyConfig)
Browse files Browse the repository at this point in the history
  • Loading branch information
ndcunningham committed Jun 15, 2023
1 parent a546d5f commit d96ada6
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 28 deletions.
6 changes: 0 additions & 6 deletions docs/generated/packages/next/executors/server.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@
"type": "string",
"description": "Hostname on which the application is served."
},
"proxyConfig": {
"type": "string",
"description": "Path to the proxy configuration file.",
"x-completion-type": "file",
"x-deprecated": "Use the built-in rewrite feature from Next.js. See: https://nextjs.org/docs/api-reference/next.config.js/rewrites"
},
"buildLibsFromSource": {
"type": "boolean",
"description": "Read buildable libraries from source instead of building them separately.",
Expand Down
18 changes: 3 additions & 15 deletions packages/next/src/executors/server/custom-server.impl.ts
Original file line number Diff line number Diff line change
@@ -1,16 +1,8 @@
import 'dotenv/config';
import {
ExecutorContext,
parseTargetString,
readTargetOptions,
runExecutor,
} from '@nx/devkit';
import { join, resolve } from 'path';
import { ExecutorContext, parseTargetString, runExecutor } from '@nx/devkit';
import { join } from 'path';

import {
NextBuildBuilderOptions,
NextServeBuilderOptions,
} from '../../utils/types';
import { NextServeBuilderOptions } from '../../utils/types';

export default async function* serveExecutor(
options: NextServeBuilderOptions,
Expand All @@ -26,10 +18,6 @@ export default async function* serveExecutor(
// Setting port that the custom server should use.
(process.env as any).PORT = options.port;

const buildOptions = readTargetOptions<NextBuildBuilderOptions>(
parseTargetString(options.buildTarget, context.projectGraph),
context
);
const projectRoot = context.projectGraph.nodes[context.projectName].data.root;

yield* runCustomServer(projectRoot, options, context);
Expand Down
6 changes: 0 additions & 6 deletions packages/next/src/executors/server/schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,6 @@
"type": "string",
"description": "Hostname on which the application is served."
},
"proxyConfig": {
"type": "string",
"description": "Path to the proxy configuration file.",
"x-completion-type": "file",
"x-deprecated": "Use the built-in rewrite feature from Next.js. See: https://nextjs.org/docs/api-reference/next.config.js/rewrites"
},
"buildLibsFromSource": {
"type": "boolean",
"description": "Read buildable libraries from source instead of building them separately.",
Expand Down
1 change: 0 additions & 1 deletion packages/next/src/utils/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ export interface NextServeBuilderOptions {
buildTarget: string;
customServerTarget?: string;
hostname?: string;
proxyConfig?: string;
buildLibsFromSource?: boolean;
keepAliveTimeout?: number;
turbo?: boolean;
Expand Down

0 comments on commit d96ada6

Please sign in to comment.