Closed djkapner closed 3 years ago
Thanks @djkapner! We also need to make some changes to incorporate some of the import syntax changes that happened from AllenSDK 2.9.0 to 2.10.0. I will add those. I'd also like to add some tests that will run on CircleCI to verify.
From your slack comment:
yep. I was just putting this here as a nudge/reminder that a new release has come out. I can stop doing that, if you have your own system going, let me know.
For AllenSDK, what we'd do is leave a branch/PR like this open, and make fixing PRs to it. When the tests on this PR pass, you're good (unless the coverage isn't complete...)
Since we're already at allensdk 2.10.2, I'm going to close this PR and start a new one to target 2.10.2 compatibility.
this may break things.