Music-and-Culture-Technology-Lab / omnizart

Omniscient Mozart, being able to transcribe everything in the music, including vocal, drum, chord, beat, instruments, and more.
https://music-and-culture-technology-lab.github.io/omnizart-doc/
MIT License
1.62k stars 98 forks source link

Latest supported Python version not tested during CI #71

Closed schneiderfelipe closed 2 years ago

schneiderfelipe commented 2 years ago

omnizart supports Python 3.8

https://github.com/Music-and-Culture-Technology-Lab/omnizart/blob/86bd36353e3eb410b5db3c18ef05f31dabfb0c7f/pyproject.toml#L26

but it's not tested during CI

https://github.com/Music-and-Culture-Technology-Lab/omnizart/blob/86bd36353e3eb410b5db3c18ef05f31dabfb0c7f/.github/workflows/general.yml#L20

Maybe the above should be changed to python-version: [3.6, 3.8], so that the oldest and newest supported versions get tested? (I believe this can be seen as a step towards partly solving #65, or at least avoiding similar issues in the future.)

BreezeWhite commented 2 years ago

Approved.