search

tom472 / mediabox

Container based media tools configuration
MIT License
431 stars 84 forks source link

general cleanup #19

Closed mnkhouri closed 6 years ago

mnkhouri commented 6 years ago

Some whitespace and other minor changes (i.e. remove backticks from commands where we aren't capturing stdout)

mnkhouri commented 6 years ago

@tom472 sorry for the delay -- I see you already beat me to most of the changes in your recent commit :)

p.s. thanks for the shout-out in the README

tom472 commented 6 years ago

Thanks for the clean-up - and thanks for the previous fixes that I merged in. Glad you saw the shout-out, I wanted to get your code adjustments in this weekend while I was fixing a couple other things. Since another PR wasn't ready yet I wanted to still give you credit for them in some way, I hope that mention in the README is acceptable.