Feedback wanted: considering removing support for NSE in inputs
Closed this issue · 2 comments
TimTaylor commented
Currently we support both quoted and unquoted inputs to the incidence function making use of {rlang} and {tidyeval} to do so. Whilst on first glance this is appealing it does complicate the code base making it more complicated to maintain and increases the chance of introducing a subtle bug. I'd be inclined to remove this feature but wanted to get some feedback first. @thibautjombart - any thoughts?
thibautjombart commented
I think this is an important discussion with likely wider application to the rest of the reconverse. I would lean towards removal, but this should be a wider consultation, as we don't want to rush this and then revert the decision :) Do you have a preferred timeline for resolving this?