🪲 remove semgrep #1249

.github/workflows/static_analysis.yaml

@@ -25,18 +25,6 @@ jobs:
       - name: Execute Test
         run: bandit -r mabel
 
-  semgrep:
-    runs-on: ubuntu-latest
-    name: SemGrep
-    steps:
-      - uses: actions/checkout@v1
-      - name: Semgrep
-        id: semgrep
-        uses: returntocorp/semgrep-action@v1
-        with:
-          config: p/r2c
-
-
   mypy:
     name: Type Checks
     runs-on: ubuntu-latest

opteryx/__init__.py

@@ -37,7 +37,7 @@
 _env_path = Path(".") / ".env"
 
 #  deepcode ignore PythonSameEvalBinaryExpressiontrue: false +ve, values can be different
-if _env_path.exists() and (dotenv is None):  # pragma: no cover  # nosemgrep
+if _env_path.exists() and (dotenv is None):  # pragma: no cover
     # using a logger here will tie us in knots
     print(f"{datetime.datetime.now()} [LOADER] `.env` file exists but `dotEnv` not installed.")
 elif dotenv is not None:  # pragma: no cover variables from `.env`")

opteryx/functions/other_functions.py

@@ -112,7 +112,7 @@ def iif(mask, true_values, false_values):
 def if_null(values, replacement):
     response = values
     for index, value in enumerate(values):
-        if value is None or value != value:  # nosec # nosemgrep
+        if value is None or value != value:  # nosec
             response[index] = replacement[index]
     return response
 

opteryx/third_party/pyarrow_ops/helpers.py

@@ -26,7 +26,7 @@ def _hash_value(val, nan=numpy.nan):
     if isinstance(val, (list, numpy.ndarray, tuple)):
         # XOR is faster however, x ^ x == y ^ y but x != y, so we don't use it
         return functools.reduce(lambda x, y: _hash_value(f"{y}:{x}", 0), val, APOLLO_11_DURATION)
-    if val != val or val is None:  # nosemgrep
+    if val != val or val is None:
         # nan is a float, but hash is an int, sometimes we need this to be an int
         return nan
     return hash(val)

tests/misc/perftest_inner_hash.py

@@ -19,7 +19,7 @@ def _hash_value(vals, nan=numpy.nan):
             ret.append(
                 functools.reduce(lambda x, y: _hash_value(f"{y}:{x}", 0), val, APOLLO_11_DURATION)
             )
-        elif val != val or val is None:  # nosemgrep
+        elif val != val or val is None:
             # nan is a float, but hash is an int, sometimes we need this to be an int
             ret.append(nan)
         else:
@@ -39,7 +39,7 @@ def _hash_value_new(vals, nan=numpy.nan):
             ret.append(CityHash64("".join(val.values())))
         elif isinstance(val, (list, tuple)):
             ret.append(CityHash64(numpy.array(val)))
-        elif val != val or val is None:  # nosemgrep
+        elif val != val or val is None:
             # nan is a float, but hash is an int, sometimes we need this to be an int
             ret.append(nan)
         else: