Skip to content

Commit

Permalink
Remove debug from dev server contract; since the logger is being inje…
Browse files Browse the repository at this point in the history
…cted it is redundant.
  • Loading branch information
joe-yeager committed Feb 15, 2024
1 parent 8d9517a commit 832179c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
6 changes: 1 addition & 5 deletions packages/cli/lib/DevServerManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ class DevServerManager {
serverInterface: DevModeInterface,
},
};
this.debug = false;
}

async iterateDevServers(callback) {
Expand Down Expand Up @@ -67,8 +66,7 @@ class DevServerManager {
}, {});
}

async setup({ components, debug, onUploadRequired, accountId }) {
this.debug = debug;
async setup({ components, onUploadRequired, accountId }) {
this.componentsByType = this.arrangeComponentsByType(components);
const { env } = getAccountConfig(accountId);
await startPortManagerServer();
Expand All @@ -77,7 +75,6 @@ class DevServerManager {
if (serverInterface.setup) {
await serverInterface.setup({
components: compatibleComponents,
debug,
onUploadRequired,
promptUser,
logger,
Expand All @@ -99,7 +96,6 @@ class DevServerManager {
if (serverInterface.start) {
await serverInterface.start({
accountId,
debug: this.debug,
projectConfig,
requestPorts,
});
Expand Down
1 change: 0 additions & 1 deletion packages/cli/lib/LocalDevManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,6 @@ class LocalDevManager {
try {
await DevServerManager.setup({
components,
debug: this.debug,
onUploadRequired: this.logUploadWarning.bind(this),
accountId: this.targetAccountId,
});
Expand Down

0 comments on commit 832179c

Please sign in to comment.