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

Add compatibility with Go 1.12 #72

Merged
merged 3 commits into from
Jun 4, 2020
Merged

Add compatibility with Go 1.12 #72

merged 3 commits into from
Jun 4, 2020

Conversation

chfast
Copy link
Collaborator

@chfast chfast commented Jun 4, 2020

No description provided.

chfast added 3 commits June 4, 2020 18:33
This actually generates better code
and brings compatibility with Go 1.12.
@chfast chfast requested a review from holiman June 4, 2020 17:00
@codecov-commenter
Copy link

Codecov Report

Merging #72 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #72   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          669       669           
=========================================
  Hits           669       669           

Copy link
Owner

@holiman holiman left a comment

Choose a reason for hiding this comment

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

LGTM

@chfast chfast merged commit 5c64e07 into master Jun 4, 2020
@chfast chfast deleted the ci branch June 4, 2020 19:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants