Conversation
…ready exists and return the intersection of the lists
|
Just for reference: Issue #38 can be closed once this PR is merged. I'm reviewing this PR now. |
|
@sreeder requested a review from @horsburgh too, so I won't go ahead and merge. Unless you want me to! |
|
Just reviewed this with @sreeder. Looks good so I'm going to merge so she can create the new release. |
Great! But can you comment on this comment/suggestion of mine, before issuing a new release?
|
|
@emiliom - we can wait, but not too long. This isn't really adding much new functionality, it's just removing a bunch of cruft. |
@horsburgh , PR #72 also was removing cruft and not adding new functionality. @ocefpaf and I have now addressed it and merged it, so we're good to go. @sreeder, go ahead and create a new release whenever you'd like. |
|
@sreeder, FYI, if you have time today to create the new odm2api release, @ocefpaf likely has time to then create the new conda and pypi packages late today. He's still traveling (in the US, on Eastern time), but allocated time today and part of tomorrow to work on our project. He was planning to work on the wofpy release today, but Don and I can't coordinate with him today, so a bit of slack probably opened up. It's your call, though. No pressure! Just wanted to make you aware of that circumstance. |
|
@emiliom Thanks for the update! |
I am planning on creating a new odm2api release version:0.6.0 when this request is approved