Skip to content

Commit 77d1cb0

Browse files
committed
chore: remove redundant runtime update comments
1 parent 094a6d3 commit 77d1cb0

2 files changed

Lines changed: 0 additions & 14 deletions

File tree

apps/webapp/app/services/projectRuntimeUpdates.server.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -85,13 +85,6 @@ export async function listCurrentProductionProjectRuntimes(scope: Scope) {
8585
);
8686
}
8787

88-
/**
89-
* Whether any project in the organization needs a Node.js runtime update in Production.
90-
*
91-
* The SQL filter mirrors `needsNodeRuntimeUpdate(runtime, runtimeVersion)`, including legacy
92-
* deployments with missing runtime metadata. Keep the two in step. Scoped to the caller's
93-
* membership so the side menu cannot report on an organization the user does not belong to.
94-
*/
9588
export async function organizationHasProjectRuntimeUpdate({
9689
organizationSlug,
9790
userId,

packages/core/src/v3/schemas/api.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -79,13 +79,6 @@ export function nodeMajor(
7979
return match ? Number(match[1]) : undefined;
8080
}
8181

82-
/**
83-
* Whether a deployment should move off the Node.js major targeted by the runtime update report.
84-
*
85-
* Older deployments can be missing either runtime field. A missing runtime historically meant
86-
* Node.js, while a missing version means we cannot prove that the deployment is on a supported
87-
* major, so both are treated as needing an update. An explicit non-Node runtime is never included.
88-
*/
8982
export function needsNodeRuntimeUpdate(
9083
runtime: string | null | undefined,
9184
runtimeVersion: string | null | undefined

0 commit comments

Comments
 (0)