Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

False positive for yoda_test_linter finding STR_CONST in $ expressions #1067

Closed
MichaelChirico opened this issue Apr 12, 2022 · 1 comment
Closed

False positive for yoda_test_linter finding STR_CONST in $ expressions #1067

MichaelChirico opened this issue Apr 12, 2022 · 1 comment
Labels
false-positive code that shouldn't lint, but does
Milestone
3.0.0

Comments

@MichaelChirico
Copy link
Collaborator

A pattern seen several times in IOHanalyzer:

https://github.com/IOHprofiler/IOHanalyzer/blob/93fdf6400f64c63c26c956086469d7a27c31ab05/tests/testthat/test_DataSetList.R#L42

expect_equal(get_FV_sample(dsl[1],12)$"run.5",13)

Here, the STR_CONST for "run.5" is at the same parse level that we check for constants in the first expression.
@MichaelChirico MichaelChirico added the false-positive code that shouldn't lint, but does label Apr 12, 2022
@MichaelChirico MichaelChirico mentioned this issue Apr 12, 2022
Merged
@MichaelChirico MichaelChirico added this to the 3.0.0 milestone Apr 12, 2022
@MichaelChirico
Copy link
Collaborator Author

closed in #1065

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
false-positive code that shouldn't lint, but does
Projects
None yet
Milestone
3.0.0
Development

No branches or pull requests
1 participant
@MichaelChirico