-
Notifications
You must be signed in to change notification settings - Fork 323
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
Renaming Variable or Function Support #7515
Conversation
engine/runtime/src/main/scala/org/enso/compiler/refactoring/IRUtils.scala
Outdated
Show resolved
Hide resolved
jobControlPlane = jobExecutionEngine, | ||
locking = locking, | ||
state = executionState, | ||
versionCalculator = Sha3_224VersionCalculator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We really should avoid using sha3, seems like an overkill, but I know that it is used in other places as well. Probably a separate ticket.
} | ||
|
||
/** | ||
* @return {@code true} if the provided name is a valid identifier of an Enso method and {@code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"of an Enso method" is this only validating method names or just symbols' names in general?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It validates names that appear on the left-hand side of the assignment. Type name can be also called an identifier (but it should be an uppercase), so I preferred to specify.
Pull Request Description
close #7389
Changelog:
refactoring/renameSymbol
request to rename locals or module methodsImportant Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.