Skip to content

Conversation

willnorris
Copy link
Collaborator

@willnorris willnorris requested a review from gmlewis June 12, 2021 23:43
@google-cla google-cla bot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Jun 12, 2021
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @willnorris !
LGTM.

@codecov
Copy link

codecov bot commented Jun 12, 2021

Codecov Report

Merging #1891 (4ae50c2) into master (edaee9a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1891   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files         105      105           
  Lines        6786     6786           
=======================================
  Hits         6627     6627           
  Misses         86       86           
  Partials       73       73           
Impacted Files Coverage Δ
github/without_appengine.go 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edaee9a...4ae50c2. Read the comment docs.

@gmlewis
Copy link
Collaborator

gmlewis commented Jun 12, 2021

Do you want to bump 1.15 to 1.16 in the GitHub workflow in this same PR, @willnorris ?

@willnorris
Copy link
Collaborator Author

Do you want to bump 1.15 to 1.16 in the GitHub workflow in this same PR, @willnorris ?

No, I actually think go1.17 getting included here is a bug (or at least undesired behavior). See my last comment here.

@willnorris willnorris merged commit 5f9e412 into google:master Jun 12, 2021
@willnorris willnorris deleted the gobuild branch June 12, 2021 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indication that the PR author has signed a Google Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Go 1.17 compiler toolchain
2 participants