Merge pull request #99 from HebaruSan/feature/empty-diff-skip-inst

Skip install step when metadata is unchanged

README.md

@@ -119,7 +119,7 @@ jobs:
 
 ## See also
 
-Validate your KSP-AVC .version files with https://github.com/DasSkelett/AVC-VersionFileValidator !
+Validate your KSP-AVC .version files with <https://github.com/DasSkelett/AVC-VersionFileValidator>!
 
 ## Contributions
 

ckan_meta_tester/ckan_meta_tester.py

@@ -189,6 +189,10 @@ def install_ckan(self, file: Path, orig_file: Path, pr_body: Optional[str], meta
         if meta_repo is not None:
             diff = ckan.find_diff(meta_repo)
             if diff is not None:
+                if len(diff) == 0:
+                    print(f'::notice file={orig_file}::Diff empty for {ckan.name} {ckan.version}, skipping install',
+                          flush=True)
+                    return True
                 with LogGroup(f'Diffing {ckan.name} {ckan.version}'):
                     print(diff, end='', flush=True)
         with LogGroup(f'Installing {ckan.name} {ckan.version}'):

tests/dummy_game_instance.py

@@ -2,7 +2,6 @@
 import unittest.util
 from unittest import TestCase
 from unittest.mock import Mock, patch, call
-from tempfile import TemporaryDirectory, TemporaryFile
 
 from ckan_meta_tester.game import Game
 from ckan_meta_tester.game_version import GameVersion
@@ -26,7 +25,7 @@ def test_dummy_game_instance_calls(self,
         mocked_run: Mock) -> None:
 
         # Arrange
-        unittest.util._MAX_LENGTH=999999999 # :snake:
+        unittest.util._MAX_LENGTH=999999999 # type: ignore # pylint: disable=protected-access
 
         # Act
         with DummyGameInstance(