Fix ruff issues

Author: peyman-salehi

.github/workflows/test.yml

@@ -91,4 +91,4 @@ jobs:
     steps:
     - uses: actions/checkout@v3
     - run: pip install --user ruff
-    - run: ruff --extend-select=C4,C9,I,PLC,PLE,PLR,U --ignore=C414,I001,PLR0913,UP007,UP032 --target-version=py38 .
+    - run: ruff check . --extend-select=C4,C9,I,PLC,PLE,PLR,U --ignore=C414,I001,PLR0913,UP007,UP032 --target-version=py38

pipeline/compressors/__init__.py

@@ -320,7 +320,7 @@ def construct_asset_path(
             os.path.dirname(source_path).replace("\\", "/"),
         )
         if self.embeddable(public_path, variant):
-            return "__EMBED__%s" % public_path
+            return f"__EMBED__{public_path}"
         if not posixpath.isabs(asset_path):
             asset_path = self.relative_path(public_path, output_filename)
         return asset_path

pipeline/packager.py

@@ -127,10 +127,10 @@ def compile(self, paths, compiler_options={}, force=False):
                 if source_storage is not None:
                     with source_storage.open(path) as source_file:
                         if self.verbose:
-                            print("Saving: %s" % path)
+                            print(f"Saving: {path}")
                         self.storage.save(path, source_file)
                 else:
-                    raise OSError("File does not exist: %s" % path)
+                    raise OSError(f"File does not exist: {path}")
         return paths
 
     def pack(self, package, compress, signal, **kwargs):
@@ -168,7 +168,7 @@ def find_source_storage(self, path):
             for short_path, storage in finder.list(""):
                 if short_path == path:
                     if self.verbose:
-                        print("Found storage: %s" % str(self.storage))
+                        print(f"Found storage: {str(self.storage)}")
                     return storage
         return None
 

tests/tests/test_compressor.py

@@ -518,7 +518,7 @@ def tearDown(self):
 
     def _test_compressor(self, compressor_cls, compress_type, expected_file):
         override_settings = {
-            ("%s_COMPRESSOR" % compress_type.upper()): compressor_cls,
+            (f"{compress_type.upper()}_COMPRESSOR"): compressor_cls,
         }
         with pipeline_settings(**override_settings):
             if compress_type == "js":

tests/tests/test_forms.py

@@ -71,8 +71,8 @@ class MyMedia(PipelineFormMedia):
         )
         self.assertEqual(MyMedia.css, media._css)
         expected_regex = [
-            r'<link href="%s"( type="text/css")? media="all" '
-            'rel="stylesheet"( /)?>' % path
+            r'<link href="{}"( type="text/css")? media="all" '
+            'rel="stylesheet"( /)?>'.format(path)
             for path in (
                 "/static/extra1.css",
                 "/static/extra2.css",
@@ -116,8 +116,8 @@ class MyMedia(PipelineFormMedia):
         self.assertEqual(MyMedia.css, media._css)
 
         expected_regex = [
-            '<link href="%s"( type="text/css")? media="all" '
-            'rel="stylesheet"( /)?>' % path
+            '<link href="{}"( type="text/css")? media="all" '
+            'rel="stylesheet"( /)?>'.format(path)
             for path in (
                 "/static/extra1.css",
                 "/static/extra2.css",

tests/tests/test_views.py

@@ -17,7 +17,7 @@ def setUp(self):
 
         self.filename = "pipeline/js/first.js"
         self.storage = staticfiles_storage
-        self.request = RequestFactory().get("/static/%s" % self.filename)
+        self.request = RequestFactory().get(f"/static/{self.filename}")
 
         default_collector.clear()