refactor: suggested changes

Signed-off-by: Pranav C <pranavxc@gmail.com>

packages/nocodb/src/db/BaseModelSqlv2.ts

@@ -1600,7 +1600,6 @@ class BaseModelSqlv2 {
                 });
 
                 const self: BaseModelSqlv2 = this;
-                // const childColumn = await colOptions.getChildColumn();
                 proto[
                   column.uidt === UITypes.Links
                     ? `_nc_lk_${column.title}`

tests/playwright/tests/utils/general.ts

@@ -37,6 +37,8 @@ function isSubset(obj, potentialSubset) {
         if (!isSubset(objValue, potentialValue)) {
           return false;
         }
+        // skip strict type check since different database returns number values in string/number type
+        // todo: revert back to strict type check once we are consistent with type
         // eslint-disable-next-line no-prototype-builtins
       } else if (!obj.hasOwnProperty(prop) || objValue != potentialValue) {
         return false;