Closed davidxia closed 1 year ago
Thanks, this mostly LGTM. Tests seem to be failing though, could you please fix?
Updated. Looks like I need to be approved each time to run the tests.
Tests still failing unfortunately :(
Sorry, fixed and made sure it passes for me locally.
Ugh, test failure is now entirely unrelated to your change, caused by actions/runner-images#6399
"Fixed" in 4ff5477 by dropping 3.6 and adding 3.11 instead. Guess we have to go with the times...
Can you rebase this PR?
Thanks for your contribution!
thank you!
with
nbstripout --keep-metadata-keys
or git config keyfilter.nbstripout.keepmetadatakeys
.closes #78