chore: update some TODO comments after workshop (#18590)

* Remove a comment that we decided was wrong * Change another comment to ask for a different thing
prisma · Apr 4, 2023 · ed52837 · ed52837
1 parent 0326571
commit ed52837
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/packages/engine-core/src/data-proxy/utils/getClientVersion.ts b/packages/engine-core/src/data-proxy/utils/getClientVersion.ts
@@ -9,7 +9,6 @@ const semverRegex = /^[1-9][0-9]*\.[0-9]+\.[0-9]+$/
 const debug = Debug('prisma:client:dataproxyEngine')
 
 async function _getClientVersion(config: EngineConfig) {
-  // TODO: now we can probably just use the client version instead
   const engineVersion = devDependencies['@prisma/engines-version']
   const clientVersion = config.clientVersion ?? 'unknown'
 

diff --git a/packages/engine-core/src/library/DefaultLibraryLoader.ts b/packages/engine-core/src/library/DefaultLibraryLoader.ts
@@ -66,7 +66,7 @@ export class DefaultLibraryLoader implements LibraryLoader {
       const incorrectPinnedPlatformErrorStr = this.platform
         ? `\nYou incorrectly pinned it to ${chalk.redBright.bold(`${this.platform}`)}\n`
         : ''
-      // TODO Improve search engine logic possibly using findSync
+      // TODO Stop searching in many locations, have more deterministic logic.
       let errorText = `Query engine library for current platform "${chalk.bold(
         this.platform,
       )}" could not be found.${incorrectPinnedPlatformErrorStr}