Remove the type from the text generated by the Extract to constant
code action
#42643
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
As described in ballerina-platform/ballerina-dev-website#8884, it is the best practice to opt out using the type symbol when declaring a constant. With this PR, the
Extract to constant
code action is modified to reflect this change.Fixes #42041
Approach
Removed appending the type symbol to the generated text edit.
Samples
Screen.Recording.2024-04-26.at.9.46.15.AM.mov
Remarks
In the ballerina-platform/ballerina-dev-website#8884, there are some scenarios in which having the type symbol in the declaration can be beneficial. However, the revised design does not consider those cases due to the following points.
Hence, it is up to the user to add the type symbol if it is necessary.
Check List