Closed james-trayford closed 1 year ago
@SamYoules this should be ready for review now, added those pesky ipynb checkpoints
to .gitignore
, fixed the git hooks script and new pre-merge check - will just need to remove tests/pre-merge/dummy.ipynb
when you're happy so the check doesn't trigger and we can merge!
Some changes to help manage the
strauss
repo better!.gitignore
file, mostly using GitHub boilerplate for python projects, but also adding WAV file ignores. This is a blanket ignore on all WAV files, with explicit exceptions for some of thedata/samples
packages bundled withstrauss
. if we add more bundles, will need to also append to.gitignore
exceptions ('unignores')bin
directory for helper scripts when working withstrauss
. first scripts set up the usefulpre-commit
hook forstrauss
developers that clears any output from ipython notebooks and re-adds them prior to commits, savig user from commiting (often large) output from.ipynb
files.ipynb
files are still present intests/pre-merge
when trying to merge withmain
, and fail if so. This prevents merging intomain
with residual pre-merge tests present, to keep themain
clean. Also add areadme.txt
in this directory to explain this.