-
Notifications
You must be signed in to change notification settings - Fork 18k
go/doc: fix detection of whole file examples #37730
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
Conversation
This PR (HEAD: 1d4092f) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/222477 to see it. Tip: You can toggle comments from me using the |
Message from Gobot Gobot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
What are the steps to get this reviewed? It has been almost a month with no updates. |
Message from Agniva De Sarker: Patch Set 1: (2 comments) Sorry for the delay. Here is a first pass. Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
Message from Gregory Petrosyan: Patch Set 1: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
Message from Agniva De Sarker: Patch Set 1: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
After CL 211357 (commit 499dc1c), hasTests and numDecl were not updated properly for function declarations with parameters, which affected the whole file example detection logic. This caused examples like package foo_test func Foo(x int) { } func Example() { fmt.Println("Hello, world!") // Output: Hello, world! } to not be detected as whole file ones.
1d4092f
to
cc71c31
Compare
This PR (HEAD: cc71c31) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/222477 to see it. Tip: You can toggle comments from me using the |
Message from Gregory Petrosyan: Patch Set 3: (1 comment) I've updated the commit message. Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
Message from Agniva De Sarker: Patch Set 3: Run-TryBot+1 Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
Message from Gobot Gobot: Patch Set 3: TryBots beginning. Status page: https://farmer.golang.org/try?commit=8d14cb06 Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
Message from Gobot Gobot: Patch Set 3: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
Message from Gregory Petrosyan: Patch Set 3: Is there anything I can do to move this CL forward? Please don’t reply on this GitHub thread. Visit golang.org/cl/222477. |
After CL 211357 (commit 499dc1c), hasTests and numDecl were not updated properly for function declarations with parameters, which affected the whole file example detection logic. This caused examples like package foo_test func Foo(x int) { } func Example() { fmt.Println("Hello, world!") // Output: Hello, world! } to not be detected as whole file ones. Change-Id: I9ebd47e52d7ee9d91eb6f8e0257511de69b2a402 GitHub-Last-Rev: cc71c31 GitHub-Pull-Request: #37730 Reviewed-on: https://go-review.googlesource.com/c/go/+/222477 Reviewed-by: Agniva De Sarker <[email protected]> Reviewed-by: Robert Griesemer <[email protected]> Run-TryBot: Agniva De Sarker <[email protected]> TryBot-Result: Gobot Gobot <[email protected]>
This PR is being closed because golang.org/cl/222477 has been merged. |
After CL 211357 (commit 499dc1c), hasTests and numDecl were not updated properly for function declarations with parameters, which affected the whole file example detection logic. This caused examples like package foo_test func Foo(x int) { } func Example() { fmt.Println("Hello, world!") // Output: Hello, world! } to not be detected as whole file ones. Fixes #38418. For #38409. Change-Id: I9ebd47e52d7ee9d91eb6f8e0257511de69b2a402 GitHub-Last-Rev: cc71c31 GitHub-Pull-Request: #37730 Reviewed-on: https://go-review.googlesource.com/c/go/+/222477 Reviewed-by: Agniva De Sarker <[email protected]> Reviewed-by: Robert Griesemer <[email protected]> Run-TryBot: Agniva De Sarker <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> (cherry picked from commit c4961dc) Reviewed-on: https://go-review.googlesource.com/c/go/+/232868 Run-TryBot: Dmitri Shuralyov <[email protected]> Reviewed-by: Gregory Petrosyan <[email protected]>
After CL 211357 (commit 499dc1c),
hasTests and numDecl were not updated properly for function
declarations with parameters, which affected the whole file
example detection logic. This caused examples like
to not be detected as whole file ones.