DocEd / fragility-index

Fragility Index Manuscript
1 stars 0 forks source link

Tidyverse #1

Open seabbs opened 4 years ago

seabbs commented 4 years ago

Hi Ed,

This looks really good and nice job for sharing the code!!

Just a little point on dependencies. It would be great if you could list which {tidyverse} packages are being used (I know it is tedious working this out and I often forget). This would make it easier for people to run this with minimal package updates! Having something like a install.R script(or using a description file) can be nice as well.

Feel free to close if I am misunderstanding something 😄

DocEd commented 4 years ago

Thanks Sam. I'm delighted that you've commented on this... I'm fairly sure this is a first for any of my code :) Good suggestions and I'll make those modifications

seabbs commented 4 years ago

Awesome!!!

Feel free to look through any/all of my code and provide some feedback 😃 - sometimes I think my code has more bugs than lines...

The tidyverse thing is annoying as it is so convenient when writing code to use it. I have it in a suggests for a CRAN package and recently got an email telling me off!