search

opensafely / research-template

The template for new research projects that use the OpenSAFELY framework.
MIT License
16 stars 13 forks source link

Update .gitignore #15

Closed CarolineMorton closed 3 years ago

CarolineMorton commented 4 years ago

add in .idea/ folder (pycharm IDE) and all contents of analysis folder except do files and other stata files to reduce chance of leakage of data

alexwalkerepi commented 4 years ago

Looks fine, though there was a hint the other day that @sebbacon was doing something with an /ouputs folder?

CarolineMorton commented 4 years ago

Yep it is failing. I think due to this issue. I could change here or wait until ready and merge @sebbacon what do you advice?

alexwalkerepi commented 4 years ago

In that case we'll need to get others to make Stata outputs etc to end up in the outputs folder too. Should we add the outputs/ folder to the .gitignore instead?

sebbacon commented 4 years ago

Ah, already done here: https://github.com/opensafely/research-template/pull/13/files

(I forgot this hasn't been merged yet)

Yes we'll have to get people to save outputs there - though I think it won't be mandatory, just a convention

CarolineMorton commented 4 years ago

@sebbacon could we merge this to make the gitignore more robust?