Skip to content

streamlink: use link_manpages#119487

Closed
cho-m wants to merge 1 commit intoHomebrew:masterfrom
cho-m:streamlink-link-manpages
Closed

streamlink: use link_manpages#119487
cho-m wants to merge 1 commit intoHomebrew:masterfrom
cho-m:streamlink-link-manpages

Conversation

@cho-m
Copy link
Member

@cho-m cho-m commented Dec 31, 2022

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

Trying again to repro error on self-hosted runner. I think I tried this before and wasn't able to in isolated PR.

If I can't repro, may just try editing test to use different video and hope it helps.

@cho-m cho-m added do not merge CI-linux-self-hosted Build on Linux self-hosted runner labels Dec 31, 2022
@BrewTestBot BrewTestBot added the python Python use is a significant feature of the PR or issue label Dec 31, 2022
@cho-m cho-m force-pushed the streamlink-link-manpages branch from 19477ed to 4451d74 Compare January 1, 2023 00:34
Comment on lines 93 to 94
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So does look like vimeo is blocked on Google Cloud self-hosted runner. Youtube seems to work. The most used test video is too long, so just grabbed the short test video in you-get.

Sadly, ok.ru may also be blocked. May keep on macOS (to test a different source) and use a different Youtube video on Linux.

@cho-m cho-m force-pushed the streamlink-link-manpages branch from 4451d74 to 8327713 Compare January 1, 2023 00:57
@cho-m cho-m added CI-no-bottles Merge without publishing bottles and removed do not merge labels Jan 1, 2023
@cho-m cho-m marked this pull request as ready for review January 1, 2023 00:57
@BrewTestBot
Copy link
Contributor

:shipit: @p-linnane has triggered a merge.

@cho-m cho-m deleted the streamlink-link-manpages branch January 1, 2023 04:45
@github-actions github-actions bot added the outdated PR was locked due to age label Feb 1, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI-linux-self-hosted Build on Linux self-hosted runner CI-no-bottles Merge without publishing bottles outdated PR was locked due to age python Python use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants