Rename workflow, reduce duplicate season checks by using a set (#144)

mizz141 · Oct 8, 2024 · 99cfc9b · 99cfc9b
1 parent 79918db
commit 99cfc9b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/.github/workflows/validate-seasons.yml b/.github/workflows/validate-seasons.yml
@@ -1,8 +1,8 @@
-name: Validate seasons
+name: TVDB
 on:
   push:
     branches:
-      - validateSeason # Remove after finalizing testing
+      - master
   pull_request:
     branches:
       - master

diff --git a/validate-tvdb.py b/validate-tvdb.py
@@ -43,14 +43,14 @@ def validateShowSeasons(showName, seasonsToFind):
     return errors
 
 # Parse temp.yaml and validate shows/seasons against TVDB
-def validateMappings():
+def validateMappings(file="temp.yaml"):
     errors = 0
-    with open("temp.yaml") as f:
+    with open(file) as f:
         mappings = yaml.safe_load(f)
         for show in sorted(mappings['entries'], key=lambda entry: (entry['title'], entry['seasons'])):
             showName = show['title']
-            seasons = [s['season'] for s in show['seasons'] if 'season' in s]
-            print("Validating: " + showName + ": " + str(seasons))
+            seasons = set([s['season'] for s in show['seasons'] if 'season' in s])
+            print("Validating: " + showName + " - Seasons " + str(seasons))
             errors += validateShowSeasons(showName, seasons)
     return errors