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

Fuzzit: use go-fuzz with go-modules support #1022

Merged
merged 1 commit into from
Aug 31, 2019

Conversation

yevgenypats
Copy link

Testing a fix which happens because go-fuzz doesn't support go-modules. current fix use Fuzzit fork which supports go-modules until this support will be integrated into go-fuzz.

#1019

cc @johanbrandhorst

@yevgenypats yevgenypats force-pushed the fuzzit-fix branch 4 times, most recently from 71334ad to 5c8d9dc Compare August 31, 2019 07:57
@codecov-io
Copy link

Codecov Report

Merging #1022 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1022   +/-   ##
=======================================
  Coverage   53.41%   53.41%           
=======================================
  Files          40       40           
  Lines        4038     4038           
=======================================
  Hits         2157     2157           
  Misses       1679     1679           
  Partials      202      202

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 2737a62...9bf0dd0. Read the comment docs.

@yevgenypats
Copy link
Author

@johanbrandhorst I think this works now. I currently use fuzzit fork until this or other commit merges into go-fuzz and I'll update again to point to original go-fuzz.

Copy link
Collaborator

@johanbrandhorst johanbrandhorst left a comment

Choose a reason for hiding this comment

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

Great, thanks for the quick fix!

@johanbrandhorst johanbrandhorst merged commit 9ab2e7e into grpc-ecosystem:master Aug 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants