PrestonStringham / DATA515-MusicGeneration

MIT License
3 stars 0 forks source link

Fix build break: Run unit tests from the top of the repo (rather than the package directory) #30

Closed carljparker closed 3 years ago

carljparker commented 3 years ago

This PR updates our unit tests so that they run from the top level of the repository instead of from the package directory.

This PR also includes flake8 fixes to our package-level __init__.py, that is, easy_music_generator/__init__.py. I incorporated these change from the following PR:

Fix flake8 errors in package-level init #29 carljparker wants to merge 1 commit into main from carljparker/flake8-in-package-init

I am discarding that PR because those changes are now included here.

As a result of recent changes, EMG now creates the output directory at the top of the repo instead of in the package directory. However, the music directory--that is, the input directory--is still in the package directory. This feels a little asymmetrical to me, but I doubt it will create any issues when the package is evaluated. Unless otherwise directed, I'll document the current behavior in the functional spec.