-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
gopls/internal/codeaction: replace all occurrences of expression (refactor.extract.variable.all) #539
base: master
Are you sure you want to change the base?
Conversation
This PR (HEAD: 4e59087) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 00f4d8a) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 2eeb2c4) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 0c4ddaf) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 2f839f4) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 78461fc) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
30abec2
to
0324b69
Compare
This PR (HEAD: 0324b69) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
37c7b7b
to
83f4dc3
Compare
This PR (HEAD: 83f4dc3) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 8581970) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 51938d2) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: f935517) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 26fbd88) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
Message from xxx Lulu: Patch Set 20: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/624035. |
This PR (HEAD: 5dcd92b) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 79bdd50) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 6cec9c5) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
b40c319
to
81352ac
Compare
This PR (HEAD: 81352ac) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 659733b) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
Message from Alan Donovan: Patch Set 25: (14 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/624035. |
fb7dd04
to
9c617a3
Compare
This PR (HEAD: 9c617a3) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
Message from xxx Lulu: Patch Set 25: (14 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/624035. |
Message from Robert Findley: Patch Set 26: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/624035. |
Message from xxx Lulu: Patch Set 26: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/624035. |
This PR (HEAD: be697e0) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: 0798b03) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
This PR (HEAD: a0a7d04) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
new test
This PR (HEAD: 8443b23) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/624035. Important tips:
|
Message from xxx Lulu: Patch Set 29: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/624035. |
This change introduces a new refactoring code action
"Extract n occurrences of expression".
The original "Extract Variable" is treated as a limited version
of "Extract n occurrences of expression", they share the same
underlying implementation. The difference is that
"extract_all" utilizes a slice of structural equal expression
by searching through the entire function body, while the limited
version simply append the expression under selection to that slice.
Also:
users will almost always perform a rename after
extraction, so this change make it more discoverable.
extract_expressions_resolve.txt.
Updates golang/go#70085