-
Notifications
You must be signed in to change notification settings - Fork 132
Code Contribution Guidelines
Brendon Watson edited this page Mar 7, 2017
·
4 revisions
- Code should note duplicate functionality. i.e. if there is already a function (code) in the repo that serves a function (purpose). You should try to merge them (without destroying backwards compatibility) - discuss your merge in the slack channel.
- Code should follow the analysis function guidelines and assume data is stored/formatted following data formatting standards.
- Before submission to this final repository, at least one other person in the lab should try it and have it work.
- Should CERTAIN people have to approve submissions?
- An open question is if we want code checks - via branches or via forks?