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

proposal: review meeting minutes #33502

Open
rsc opened this issue Aug 6, 2019 · 3 comments

Comments

@rsc
Copy link
Contributor

commented Aug 6, 2019

The proposal review group meets regularly (roughly weekly) to review pending proposal issues and move them along in the proposal process.

Review consists primarily of checking that discussion is ongoing, commenting on issues to move discussion along, summarizing long discussions, CC’ing experts, and accepting or closing proposals when the discussion on the GitHub issue has reached a clear consensus about the outcome.

Review also includes closing proposals that are untenable (for example, because the changes are backwards-incompatible or violate key design goals of the language or packages).

This meta-issue records minutes of the proposal review meetings as issue comments, so that they can be cross-linked easily with the relevant issues. (This meta-issue is for minutes only; comments that are not meeting minutes will be deleted.)

@golang golang locked and limited conversation to collaborators Aug 6, 2019

@rsc rsc changed the title placeholder proposal: review meeting minutes Aug 6, 2019

@gopherbot gopherbot added this to the Proposal milestone Aug 6, 2019

@gopherbot gopherbot added the Proposal label Aug 6, 2019

@rsc

This comment has been minimized.

Copy link
Contributor Author

commented Aug 6, 2019

2019-08-06 / @rsc, @griesemer, @andybons

  • #32816 cmd/fix: automate migrations for simple deprecations
    • discussion ongoing (no action taken)
  • #33388 cmd/go: add build tags for 32-bit and 64-bit architectures
    • retitled, commented
  • #33203 cmd/go: add package search functionality
    • commented
  • #30240 cmd/go: automatic vendoring in module mode
    • discussion ongoing
  • #33375 cmd/go: introduce go.gen file for //go:generate comments
    • commented
  • #33281 crypto engines
    • labeled Proposal-Crypto; discussion ongoing
  • #33430 crypto: crypto.Hash implement fmt.Stringer and export crypto.maxHash
    • labeled Proposal-Crypto, commented
  • #32779 encoding/json: memoize strings during decode?
    • commented, commented on CL
  • #33473 errors: generalize As to all types
    • retitled, commented
  • #33457 image: add generic metadata support for jpeg, gif, png
    • retitled, cc’ed expert
  • #33454 log: modify `Logger` struct, make it supports multi output
    • closed (backwards-incompatible change; easy implementation outside standard library; no evidence of widespread need)
  • #33498 net/url: add (Values).AddAll(key string, values []string)
    • retitled, commented
  • #29982 reflect: add constants for size of int, bool, etc.
    • summarized
  • #33449 text/template: make "template" action accept dynamic name as argument
    • commented, cc’ed expert
  • #33466 x/tools: tool to audit diffs in dependencies
    • discussion ongoing
  • #33352 x/tools/cmd/gopls: provide config option for hover documentation placement
    • discussion ongoing
@andybons

This comment has been minimized.

Copy link
Member

commented Aug 13, 2019

2019-08-13 / @rsc, @griesemer, @andybons, @spf13

  • #25530 cmd/go: secure releases with transparency log
    • commented
    • likely accept; last call for comments
  • #32405 errors: simplified error inspection
    • commented
    • likely decline; last call for comments
  • #31572 net/http: add constant for content type
    • commented
    • likely decline; last call for comments
  • #32456 net/url: add FromFilePath and ToFilePath
    • asked for design doc
    • put on hold for design doc
  • #29696 runtime: add way for applications to respond to GC backpressure
    • asked owners for status
  • #32115 runtime: KeepAlive should perhaps keep things from being moved
    • asked owners for feedback
  • #29165 text/template: allow io.Reader as substitution value
    • suggested design doc
  • #30984 x/net/nettest: add MakeLocalPipe to construct local listener and connections
    • commented
    • pinged owners of package
@andybons

This comment has been minimized.

Copy link
Member

commented Aug 20, 2019

2019-08-20 / @rsc, @griesemer, @andybons, @ianlancetaylor

  • #33702 deterministic execution
    • closed (no concrete change to evaluate)
  • #25530 cmd/go: secure releases with transparency log
    • no final comments; accepted 🎉
  • #33388 cmd/go: add build tags for 32-bit and 64-bit architectures
    • commented
  • #33203 cmd/go: add package search functionality
    • commented
  • #33375 cmd/go: introduce go.gen file for //go:generate comments
    • commented
    • likely decline; last call for comments
  • #32405 errors: simplified error inspection
    • no final comments; declined
  • #31572 net/http: add constant for content type
    • no final comments; declined
  • #33498 net/url: add (Values).AddAll(key string, values []string)
    • commented
    • likely decline; last call for comments
  • #33670 proposal: identify large changes & add more process
    • discussion ongoing
  • #33136 reflect: add Value.SetZero
    • commented
  • #33508 x/sys/cpu -- add support for ARM
    • commented
    • asked owners for feedback
  • #32153 x/sys/unix: support for all fields of Kevent_t, EpollEvent
    • commented
    • reduced scope
    • retitled
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants
You can’t perform that action at this time.