I made a menu-driven CLI frontend for yt-dlp. Feedback is welcomed. (github.com)

submitted 4 days ago* (last edited 3 days ago) by AstroLightz@lemmy.world to c/python@programming.dev

12 comments fedilink hide all child comments

I made this Python menu-driven CLI script for yt-dlp. It guides you through a series of menus to download content from a YouTube URL.

If anyone is interested, feedback would be greatly appreciated as only I have tested this so far, and I want to make sure it works for other people's systems.

If you encounter any issues or bugs, let me know so I can fix them!

Edit: I should mention: This is not a pipx package. That was never the intention developing this initially. I might make it a package in the future, but for now, it is just a project directory.

top 12 comments

sorted by: hot top controversial new old

[-] vimmiewimmie@lemm.ee 5 points 3 days ago

I like the thought behind this. I've considered something similar for yt-dlp. Thanks for putting it out there and keep it up!

[-] dumblederp@aussie.zone 1 points 3 days ago

I use tartube already.

[-] rimu@piefed.social 3 points 4 days ago

I tried to run it but it wouldn't on the version of python I have.

If you could bring yourself to use if .. elif .. etc instead of match then it could run it on much older versions of python.

[-] pupbiru@aussie.zone 5 points 4 days ago

but then they wouldn’t be using match

[-] logging_strict@programming.dev -3 points 3 days ago* (last edited 3 days ago)

Using match is virtue signaling that have no intention of creating a working package.

What's next on the list of crap could all live without?

exceptiongroup
colorama
six
pyyaml

[-] AstroLightz@lemmy.world 1 points 3 days ago

What version of Python were you using?

[-] rimu@piefed.social 1 points 3 days ago

3.9

[-] AstroLightz@lemmy.world 1 points 2 days ago

I'm testing different Python versions to see what the minimum is for yt-dlp-adv is. I know 3.10 is when they added match-case, so 3.9 isn't supported. Will update README with minimum working version after I'm done testing.

[-] logging_strict@programming.dev -1 points 3 days ago

Why? It's not a package? There are no tests or anything else. It's held together with duct tape, hope, and good intentions. So of course it'll not work as intended.

[+] logging_strict@programming.dev -7 points 3 days ago* (last edited 3 days ago)

User friendly LOL!

If you encounter any issues or bugs, let me know so I can fix them!

You shoulda lead with that. I love the humor. Can't stop laughing,

The entire project is a bug LOL!

Where to start?

There is no packaging at all

Start with a pyproject.toml and work from there.

no tests

Everything is a bug until it's got test coverage.

screenshots

In the .github folder?! That's gotta be a 1st

no dev environment

Expecting pre-commit as well as isort, flask, black, and mypy

print statements galore

Looked into requirements.txt expecting to find a console UI framework. There is none!

A pattern has emerged that many Python coders have spent not enough to no time learning packaging, dev toolchain, and CI/CD publishing. When asking folks to test your work they'll be expecting a published package, not a series of amateurish scripts and a bash install script.

Should write an advertisement

Please someone skilled at console UI and packaging please please please help in a paid position.

Can confidently say, you need help.

Not writing more features the OP is good at that. Just packaging and swapping out the prehistoric console UI with a modern console UI framework.

[-] rimu@piefed.social 6 points 3 days ago

Dunking on noobs is not cool.

[-] logging_strict@programming.dev -3 points 3 days ago* (last edited 3 days ago)

You yourself are a victim

Having good intentions you tried and found out the hard way that in fact packaging does matter.

You were tricked.

I looked at it, recognized the flaming turd being thrown at the proverbial wall, and dodged.

That is our job when doing code reviews and offering advice. Be kind up to the point where being honest is unavoidable.

A series of scripts does not make a package. Have to put our collective foot down; follow Nancy Reagon's advice, Just say no!

This project cannot be helped. It needs a complete rewrite.

Having minimal expectations is not being mean to noobs. Not getting anywhere in the ballpark of minimal expectations is being mean to potential users ...