Skip to content

Fix: Workflow exits successfully when yt-dlp is up to date#126

Merged
athphane merged 2 commits into
masterfrom
copilot/fix-github-actions-workflow
Feb 12, 2026
Merged

Fix: Workflow exits successfully when yt-dlp is up to date#126
athphane merged 2 commits into
masterfrom
copilot/fix-github-actions-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 12, 2026

The yt-dlp version check workflow fails when versions are already current, despite functioning correctly. The Python comparison script exits with code 1 to signal "no update needed," which bash interprets as failure.

Changes

  • Add explicit exit 0 after setting needs_update=false in the version comparison step
elif [ $EXIT_CODE -eq 1 ]; then
  echo "needs_update=false" >> $GITHUB_OUTPUT
  echo "yt-dlp is up to date"
  exit 0  # Ensure step succeeds when no update needed

This ensures the workflow succeeds when yt-dlp is current, while actual comparison errors (exit code 2) still fail the workflow.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: athphane <13810742+athphane@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix return codes in yt-dlp version check script Fix: Workflow exits successfully when yt-dlp is up to date Feb 12, 2026
Copilot AI requested a review from athphane February 12, 2026 19:06
@athphane athphane marked this pull request as ready for review February 12, 2026 19:06
@athphane athphane merged commit 7ef8d0b into master Feb 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants