These are chat archives for biojs/biojs

30th
May 2018
rowlandm
@rowlandm
May 30 2018 22:52
Just a couple of suggestions around Pull Requests (PR)
It's a good idea to keep doing pull requests for small stuff as opposed to one big PR. It's very hard to review one big one.
It also means you get more feedback early on if you haven't got your high level algorithm right
One trick I have learned is to ask students to do a pull request with only comments so that I can see how the students see the algorithm of what they need to do without getting into details.
Sometimes they create new files and put comments in those so I can see what the layout might look like
and then put in comments for functions and where those functions will be called
so I can see an overview in the one pull request very easily (Like in 5 minutes)
As soon as you get actual code it can get really hard to see that overview