-
Notifications
You must be signed in to change notification settings - Fork 395
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
design: add proposal for parameterized generic array sizes #33
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
1ac6b87
to
58cbf07
Compare
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
58cbf07
to
c3ddb0e
Compare
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@googlebot I signed it! |
This PR (HEAD: c3ddb0e) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/proposal/+/301290 to see it. Tip: You can toggle comments from me using the |
Message from Go Bot: 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 Please don’t reply on this GitHub thread. Visit golang.org/cl/301290. |
Message from Ian Lance Taylor: Patch Set 1: (6 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/301290. |
c3ddb0e
to
8e3bc88
Compare
This PR (HEAD: 8e3bc88) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/proposal/+/301290 to see it. Tip: You can toggle comments from me using the |
Message from Andrew Werner: Patch Set 2: (7 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/301290. |
Detailed proposal for golang/go#44253. Change-Id: I9e37363a4d5c41de100def1222d50ff6ad09d078 GitHub-Last-Rev: 8e3bc88 GitHub-Pull-Request: #33 Reviewed-on: https://go-review.googlesource.com/c/proposal/+/301290 Reviewed-by: Ian Lance Taylor <iant@golang.org>
This PR is being closed because golang.org/cl/301290 has been merged. |
Detailed proposal for golang/go#44253.