catalystneuro / mease-lab-to-nwb

MIT License
3 stars 2 forks source link

add basic CI & update conda env dependencies #44

Closed lkeegan closed 3 years ago

lkeegan commented 3 years ago
alejoe91 commented 3 years ago

@lkeegan is this ready to merge?

As a sidenote, we think that the best pipeline would be that you make your own fork of the repo, so you can continue working on it 'independently', i.e. without needing our approval to make changes. What do you think?

lkeegan commented 3 years ago

@alejoe91 yes this should be ready to merge.

And sure, I can make changes to my fork without also submitting PRs here if that woudl work better for you - either way is fine by me.

alejoe91 commented 3 years ago

@alejoe91 yes this should be ready to merge.

Merging then!

And sure, I can make changes to my fork without also submitting PRs here if that woudl work better for you - either way is fine by me.

No need to make incremental PRs at this stage in my opinion. :)