Conversation
|
I will pull in the new changes from #21 and then do a PR |
|
@KonradUdoHannes Done merging the changes of #21 |
|
I'm still not sure whether we actually need dashboard in the client, but I think we can still use the PR to address a weakness in our project setup. Right now we potentially get a whole bunch of tiny issues from formatting, linting and line ending when merging our different PRs. At the same time these issues are hardly worth discussing by themselves. In order to avoid them in the first place I introduced pre-commit hooks (already merged in main). These will ensure that every commit automatically passes certain requirements (and they also help fix a lot of the issues). I would suggest that you bring the branch up to date with main and the try to set up pre-commit as described in the readme. Generally this is a very useful tool, not just for R but also for a lot of other programming languages. Let me know if there is any trouble. |
No description provided.