Remove some todos.

powersync-ja · Jul 4, 2024 · ed361a9 · ed361a9
1 parent 75fe43c
commit ed361a9
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/packages/sync-rules/src/sql_filters.ts b/packages/sync-rules/src/sql_filters.ts
@@ -554,7 +554,6 @@ export class SqlTools {
         }
       } satisfies RowValueClause;
     } else if (argsType == 'param') {
-      // TODO: make sure this is properly unique & predictable
       const argStrings = argClauses.map((e) => {
         if (isParameterValueClause(e)) {
           return e.key;

diff --git a/packages/sync-rules/test/src/sync_rules.test.ts b/packages/sync-rules/test/src/sync_rules.test.ts
@@ -407,7 +407,6 @@ bucket_definitions:
       }
     ]);
 
-    // TODO: Deduplicate somewhere
     expect(
       rules.evaluateRow({ sourceTable: ASSETS, record: { id: 'asset2', description: 'test', role: 'normal' } })
     ).toEqual([