Shopify · sle-c · Jan 13, 2025 · Jan 11, 2025 · Jan 11, 2025
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -2,15 +2,15 @@
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
  - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v3.4.0
+    rev: v5.0.0
    hooks:
      - id: end-of-file-fixer
      - id: trailing-whitespace
  - repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 24.10.0
    hooks:
      - id: black
  - repo: https://github.com/PyCQA/pylint
-    rev: v2.15.8
+    rev: v3.3.3
    hooks:
      - id: pylint
diff --git a/scripts/shopify_api.py b/scripts/shopify_api.py
@@ -128,7 +128,7 @@ def add(cls, connection):
        if os.path.exists(filename):
            raise ConfigFileError("There is already a config file at " + filename)
        else:
-            config = dict(protocol="https")
+            config = {"protocol": "https"}
            domain = input("Domain? (leave blank for %s.myshopify.com) " % (connection))
            if not domain.strip():
                domain = "%s.myshopify.com" % (connection)

diff --git a/shopify/api_access.py b/shopify/api_access.py
@@ -14,7 +14,6 @@ class ApiAccessError(Exception):


 class ApiAccess:
-
    SCOPE_DELIMITER = ","
    SCOPE_RE = re.compile(r"\A(?P<unauthenticated>unauthenticated_)?(write|read)_(?P<resource>.*)\Z")
    IMPLIED_SCOPE_RE = re.compile(r"\A(?P<unauthenticated>unauthenticated_)?write_(?P<resource>.*)\Z")

diff --git a/shopify/mixins.py b/shopify/mixins.py
@@ -24,7 +24,7 @@ def add_metafield(self, metafield):
        if self.is_new():
            raise ValueError("You can only add metafields to a resource that has been saved")

-        metafield._prefix_options = dict(resource=self.__class__.plural, resource_id=self.id)
+        metafield._prefix_options = {"resource": self.__class__.plural, "resource_id": self.id}
        metafield.save()
        return metafield


diff --git a/shopify/session.py b/shopify/session.py
@@ -54,7 +54,7 @@ def __init__(self, shop_url, version=None, token=None, access_scopes=None):
        return

    def create_permission_url(self, scope, redirect_uri, state=None):
-        query_params = dict(client_id=self.api_key, scope=",".join(scope), redirect_uri=redirect_uri)
+        query_params = {"client_id": self.api_key, "scope": ",".join(scope), "redirect_uri": redirect_uri}
        if state:
            query_params["state"] = state
        return "https://%s/admin/oauth/authorize?%s" % (self.url, urllib.parse.urlencode(query_params))
@@ -69,7 +69,7 @@ def request_token(self, params):
        code = params["code"]

        url = "https://%s/admin/oauth/access_token?" % self.url
-        query_params = dict(client_id=self.api_key, client_secret=self.secret, code=code)
+        query_params = {"client_id": self.api_key, "client_secret": self.secret, "code": code}
        request = urllib.request.Request(url, urllib.parse.urlencode(query_params).encode("utf-8"))
        response = urllib.request.urlopen(request)