nsh87 / receptormarker

Source for 'receptormarker' package for R: antibody receptor and phenotypic marker analysis
http://receptormarker.com
BSD 2-Clause "Simplified" License
4 stars 9 forks source link

[Wrong Merge Branch] Minor fixes: lintr, Readme, and change where Slack notifications go #51

Closed nsh87 closed 8 years ago

nsh87 commented 8 years ago

This contains some miscellaneous updates:

  1. There was a lintr error that was stopping the package from building. Turns out lintr was picking up a comment in the code that contained code (i.e. it was commented code). lintr didn't like that and said commented code should be removed, so I just told lintr to ignore that line.
  2. Developers joining the project and following the instructions on the Readme would not have muscle installed and so loading the package would fail. Added into the Readme that they should install muscle from Bioconductor.
  3. We created a new Slack team, and Travis notifications should go there rather than our old team/channel. Changed the notification key to make this happen.