Remove audio format postprocessing (#12)

* Remove audio format postprocessing * Add test, pre-release, release jobs * PACKAGE -> PACKAGE_NAME
carlthome · Oct 22, 2023 · e20ca27 · e20ca27
1 parent 66179f5
commit e20ca27
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -12,6 +12,9 @@ on:
     branches:
       - main
 
+env:
+  PACKAGE_NAME: audioscrape
+
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
   cancel-in-progress: true
@@ -21,17 +24,23 @@ permissions:
 
 jobs:
   test:
+    if: ${{ github.event_name == 'pull_request' }}
+    uses: carlthome/workflows/.github/workflows/python.yaml@main
+    with:
+      package-name: $PACKAGE_NAME
+
+  pre-release:
     if: ${{ github.event_name == 'push' && startsWith(github.ref, 'refs/heads/main') }}
     uses: carlthome/workflows/.github/workflows/python.yaml@main
     with:
-      package-name: audioscrape
+      package-name: $PACKAGE_NAME
       package-index-name: TestPyPI
       package-index-url: https://test.pypi.org/legacy
 
   release:
     if: ${{ github.event_name == 'release' && github.event.action == 'published' }}
     uses: carlthome/workflows/.github/workflows/python.yaml@main
     with:
-      package-name: audioscrape
+      package-name: $PACKAGE_NAME
       package-index-name: PyPI
       package-index-url: https://pypi.org
diff --git a/audioscrape/youtube.py b/audioscrape/youtube.py
@@ -63,13 +63,6 @@ def scrape(query, include, exclude, quiet, verbose, overwrite, limit):
             "writeinfojson": True,
             "writethumbnail": True,
             "writedescription": True,
-            "postprocessors": [
-                {
-                    "key": "FFmpegExtractAudio",
-                    "preferredcodec": "vorbis",
-                    "preferredquality": "192",
-                }
-            ],
         }
         ydl = yt.YoutubeDL(download_options)