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.

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

Discovery: RenameLocalVariablesToCamelCase does not rename all applicable names. #62

Open
traceyyoshima opened this issue Jun 28, 2023 · 0 comments

Comments

@traceyyoshima
Copy link
Contributor

Issue:
Namespace and the location of valid identifiers may differ between languages. RenameLocalVariablesToCamelCase contains logic limiting changes to valid Java locations to prevent unexpected results. JavaScript allows properties to be declared with keywords that change the applicable scope of the named variable let vs. var. Additionally, some JavaScript trees contain identifiers that should be renamed that would not be valid in Java. The "invalid" identifiers are not renamed, which results in incorrect code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

2 participants