midi2tones
midi2tones copied to clipboard
Add support for providing full filename on command line
This fixes an issue I had using the tool with filename completion by allowing the user to provide the .mid extension when specifying the base filename. It will strip that off before processing.
BTW, I'd appreciate it if you could label this PR as "hacktoberfest-accepted" even if you don't want to merge it so it will count towards my goal for the month.
midi2tones is a fork of miditones. For things other than changes that @LenShustek has chosen not to incorporate into miditones, I try to keep them in sync (although I'm a bit behind on that).
For this reason, I suggest that you submit an equivalent PR for miditones. If it's accepted there, I'll port it into midi2tones or merge a submitted PR that does so. If it's not accepted, I'll still consider applying this PR, as I think it's a useful enhancement.
However,
-
I prefer that the commit message follow the "50/72" guidelines, as suggested by Tim Pope. The first line should be no more than 50 characters.
-
Consider also stripping off a
.MIDextension (in uppercase) to better accommodate case insensitive file systems. -
The new version should be 1.1.0, since this is a backwards compatible enhancement, not a bug fix.
I've labelled this PR "hacktoberfest-accepted", as requested.
Ah, I'd assumed that the original repo wasn't being maintained, since I only saw your changes, but now I see its had activity in 2019. I'll reformat my commit message and see about getting the PR accepted there. The idea for supporting ".MID" also is good, so I'll add that to my branch. Thanks!
Created PR for Len at https://github.com/LenShustek/miditones/pull/23