-
Notifications
You must be signed in to change notification settings - Fork 37
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
No comments in code #13
Comments
No comments in source code is a good thing. |
By the way, I cloned this repo (and the git submodule), executed it, and got the error message: "Failed to connect to Aria2", but I don't see any documentation on how to get the repo working. |
I found the solution to make the app work, it would be nice to add this to the docs: ## Project Setup
Next to the "`YoutubeApp.exe`" main app, create a folder named "utils" and get these 3 files from the internet:
- "aria2c.exe" from https://github.com/aria2/aria2/releases (look for the releases file download `aria2-1.37.0-win-64bit-build1.zip`).
- "yt-dlp.exe" from https://github.com/yt-dlp/yt-dlp/releases (look for the `yt-dlp.exe` file)
- "ffmpeg.exe" from https://github.com/BtbN/FFmpeg-Builds/releases (for example the `ffmpeg-master-latest-win64-gpl.zip` and extract the `ffmpeg.exe`)
|
Hard disagree. Good comments help understanding and make code more readable. Although I admit, this is a highly debated topic in the community, it would seem that most major projects also use comments. Also bringing up a totally different problem in this issue is inappropriate, please open a new issue for those instead of discussing them here. |
This issue is too generic and open ended, what amount of comments, where, about what in particular are you expecting? About comments, for sure "good" comments are great specially when they explain the questions "why" the code is that way, but sometimes there is just nothing worth commenting down, I don't know if it is the case for this repo or not, but the issue sounds like saying "No comments >> therefore the code is hard to read", which is not necessarily true. |
There are very few comments in the source code, making it hard to read. Do you plan to add more? Also some overview explanation about the software architecture would be very welcome.
Great project otherwise!
The text was updated successfully, but these errors were encountered: