search

jhump / protoreflect

Reflection (Rich Descriptors) for Go Protocol Buffers
Apache License 2.0
1.35k stars 172 forks source link

GitHub actions: Enable CI and CIFuzz GitHub action #541

Open DavidKorczynski opened 1 year ago

DavidKorczynski commented 1 year ago

This adds a .github/workflows folder to enable Github actions. Also adds CIFuzz workflow action to have fuzzers build and run on each PR.

This is a service offered by OSS-Fuzz where Protoreflect already runs (https://github.com/google/oss-fuzz/tree/master/projects/protoreflect). CIFuzz can help detect regressions, move fuzzing more into the development cycle, and catch fuzzing build issues early. CIFuzz has a variety of features (see the URL above). In the current PR the fuzzers gets build on a pull request and will run for 300 seconds.

jhump commented 1 year ago

@DavidKorczynski, thanks for this PR! Sorry it took me so long to reply.

I was actually about to update the OSS Fuzz configuration to instead directly run the fuzz testing against https://github.com/bufbuild/protocompile, which is the new parser/compiler implementation that now backs the desc/protparse package in this repo (as of #354 which was merged about two weeks ago).

So I expect to apply this config over in that repo instead of merging this.