Skip to content
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

doc/go1.17: document reflect changes for Go 1.17 #46019

Closed
gopherbot opened this issue May 6, 2021 · 2 comments
Closed

doc/go1.17: document reflect changes for Go 1.17 #46019

gopherbot opened this issue May 6, 2021 · 2 comments

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented May 6, 2021

As of filing this issue, the Go 1.17 draft release notes contained the following HTML:

reflect

TODO: https://golang.org/cl/266197: add Method.IsExported and StructField.IsExported methods

TODO: https://golang.org/cl/281233: add VisibleFields function

TODO: https://golang.org/cl/284136: panic if ArrayOf is called with negative length


This TODO needs to be resolved by July 1 so that we can have complete release notes for Beta 1.

A sequence of steps to resolve this issue may be:

  1. Come up with a draft release note entry in this issue, then move the issue to NeedsFix state.
  2. Send a CL with "doc/go1.17:" prefix and include "For #44513. Fixes #46019." in the body.
@gopherbot
Copy link
Author

@gopherbot gopherbot commented May 7, 2021

Change https://golang.org/cl/318011 mentions this issue: doc/go1.17: fill in TODO for reflect package

gopherbot pushed a commit that referenced this issue May 7, 2021
Updates #46019

Change-Id: I3025927d949ff72535542e89b83dd830e969c255
Reviewed-on: https://go-review.googlesource.com/c/go/+/318011
Trust: Joe Tsai <joetsai@digital-static.net>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
@gopherbot
Copy link
Author

@gopherbot gopherbot commented May 27, 2021

Change https://golang.org/cl/323069 mentions this issue: doc/go1.17: document reflect changes

@gopherbot gopherbot closed this in ab2ef4a May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant