-
Notifications
You must be signed in to change notification settings - Fork 755
feat(Dockerfile): add a wrapper for yt-dlp to use node runtime #1687
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
06c0cff to
9cc35c1
Compare
|
build for armv6/armv7 is now removed due to unsupported by deno |
|
Based on the log from #1684, deno should be unnecessary. Mark as draft for now. |
Wait until next release is made. |
Signed-off-by: Tianling Shen <[email protected]>
|
replaced |
pan93412
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
The docker PAT seems expired, could you please check & renew it? |
will update it soon. |
yt-dlp uses deno as the default js runtime, add a wrapper to force use node.js.
Fixed #1684