-
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/highlight: DocumentHighlight for format strings #555
Conversation
da0e15e
to
32f433e
Compare
This PR (HEAD: 32f433e) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
32f433e
to
6317eaf
Compare
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from xxx Lulu: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
This PR (HEAD: 6317eaf) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
6317eaf
to
d894536
Compare
This PR (HEAD: d894536) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
Message from Alan Donovan: Patch Set 3: (14 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Alan Donovan: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Robert Findley: Patch Set 3: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
86c4dc8
to
d7d075e
Compare
This PR (HEAD: d7d075e) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
d7d075e
to
c88d0d1
Compare
Message from xxx Lulu: Patch Set 4: (15 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
This PR (HEAD: c88d0d1) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
Message from xxx Lulu: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
c88d0d1
to
a6c7d79
Compare
This PR (HEAD: a6c7d79) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
Message from Alan Donovan: Patch Set 6: (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
b5bbcbe
to
f8fc938
Compare
This PR (HEAD: f8fc938) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
Message from xxx Lulu: Patch Set 7: (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
7d3b736
to
e1ba409
Compare
This PR (HEAD: e1ba409) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
e1ba409
to
d15a3b3
Compare
Message from Alan Donovan: Patch Set 7: Code-Review+2 Commit-Queue+1 (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Go LUCI: Patch Set 7: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-01-14T19:50:28Z","revision":"1fe498ed45496528db5fd794cdcef563816e507f"} Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
This PR (HEAD: d15a3b3) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/tools/+/642095. Important tips:
|
Message from Go LUCI: Patch Set 7: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Robert Findley: Patch Set 8: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from xxx Lulu: Patch Set 8: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Robert Findley: Patch Set 9: Code-Review+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Robert Findley: Patch Set 9: Auto-Submit+1 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Go LUCI: Patch Set 9: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-01-14T20:17:21Z","revision":"afa9f5779dc68ad1032cf32dad0829fca1765d1d"} Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
Message from Alan Donovan: Patch Set 9: Auto-Submit+1 Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/642095. |
This CL introduces functionality for highlighting printf-style directives and their associated variadic arguments. Also fix some comments/names in CL 623156 Updates golang/go#70050 Change-Id: I1c4e6678317aa8cf522f41765f5a6600793f3746 GitHub-Last-Rev: d15a3b3 GitHub-Pull-Request: #555 Reviewed-on: https://go-review.googlesource.com/c/tools/+/642095 LUCI-TryBot-Result: Go LUCI <[email protected]> Auto-Submit: Robert Findley <[email protected]> Auto-Submit: Alan Donovan <[email protected]> Reviewed-by: Alan Donovan <[email protected]> Reviewed-by: Robert Findley <[email protected]>
This PR is being closed because golang.org/cl/642095 has been merged. |
This CL introduces functionality for highlighting
printf-style directives and their associated variadic arguments.
Also fix some comments/names in CL 623156
Updates golang/go#70050