Tom Jemmett

oh, before changing that, wait for #176 to merge

or, at least start your PR off of #176 to begin with. Otherwise, you are going to be in merge conflict hell :-)

one thing I think we should be careful of is to try to follow the Unix philosophy (do one thing, do it right). This is designed to do the NHS...

The images are within vignettes and the readme only - I'm not sure any of these currently have alt text added. The two ways of doing this are: 1. if...

so originally it did sort of do this by introducing multiple columns (one per rule), then an overall column. This wasn't particularly easy to test, so each rule was turned...

Do we want to annotate the data frame, or the plots? I think the latter. I would argue against adding additional arguments where possible as it starts to bloat the...

I think you are probably right - these are much nicer descriptions! If you want to create a PR I will approve and merge :-) Might be worth having both...

Sorry, missed this! I thought it could be bookdown, though it may increase the effort we put on ourselves for contributions. I would be happy to convert it to bookdown...

you could treat this as a continous function by pivotting the dates into a single column, then using `referral_date` as +1, `discharge_date` as -1. If you then arrange on the...

separate issue for legend in #177 🙂