Support non-syntactic names in object_overwrite_linter()
#2346
Comments
AshesITR
added a commit
that referenced
this issue
Nov 23, 2023
MichaelChirico
added a commit
that referenced
this issue
Nov 24, 2023
…2347) * fix detection of non-syntactic names in `object_overwrite_linter()` fixes #2346 * fix non-syntactic string assignments * actually fix it * remove unnecessary startsWith() calls * avoid escape * move to separate test_that() --------- Co-authored-by: Michael Chirico <michaelchirico4@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This will be obsolete with #2344 I think, since I use
get_r_string()there so overwrites of non-syntactic names should also be caught.This reminds me that could use some tests.
Originally posted by @AshesITR in #2342 (comment)
The text was updated successfully, but these errors were encountered: