Closed aln363 closed 1 month ago
This is looking really really good to me! I think this is going to be super helpful for people diving into FFmpeg for the first time, and I really appreciate the "new user's perspective" that this brings.
Overall, I don't have any larger structural comments, I think this looks good. Other maintainers, what do you think?
I see a couple of "nits" (very minor typo issues) and I'll address those comments in a code review.
Trial by fire! Thank you SO much for this contribution and don't take my comments negatively, I'm thrilled to see this contribution!
Circling back to this! @aln363 - did you have a chance to look at @ablwr comments?
Ping @aln363 : I’d like to open a few PRs that might conflict with yours. That’s why I would be happy if this could be finalised.
Hi team! Where are we at with this PR currently? I was going to make some time to review, but I see that there are already a bunch of pending comments. @aln363, how are things looking at your end? :relaxed:
ping
Closing this - feel free to reopen in future if ready to merge
This fork comes out of a final project for NYU MIAP's digital literacy course, it is designed for beginner archivists using ffmpeg who may encounter errors they don't know how to solve.