These are chat archives for ROBelgium/MSNoise

12th
Oct 2017
Damian Kula
@heavelock
Oct 12 2017 10:42
Hi guys!
I've just started a PhD at l'EOST, Unistra and I'm slowly starting to get into the MSNoise
Don't you mind If I refactor code a bit and write some unit tests? :)
Tobias Megies
@megies
Oct 12 2017 10:49
I thnk @ThomasLecocq is happy about new collaborators.. in any case, best write tests first and refactor later, that's the right sequence.. ;-)
Damian Kula
@heavelock
Oct 12 2017 10:50
Yeah. sure :)
Thomas Lecocq
@ThomasLecocq
Oct 12 2017 11:32
Hi @heavelock
you might want to check the new_cc branch too, it's WIP but definitively going to be merged at some point
what /where do you want to refactor ?
Thomas Lecocq
@ThomasLecocq
Oct 12 2017 11:42
I've nothing against it, but maybe create an Issue with a list of things you plan, so we can discuss them and keep track when you'll PR those changes
Damian Kula
@heavelock
Oct 12 2017 13:01
At this moment, besides of a PR I submitted, I'm thinking about refactoring test suite to use the tempdir instead of real dirs. I will create issue to continue discussion in a structured manner.