-
-
Notifications
You must be signed in to change notification settings - Fork 903
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
maintain oss-fuzz target building upstream #3093
maintain oss-fuzz target building upstream #3093
Conversation
For example: fuzzy-boiii23a/oss-fuzz@9d65673 |
Other than the missing newline, LGTM. |
Heh, now it has an extra newline, but that's fine. (Some tools treat the POSIX requirement that all lines in a text file end in a newline seriously was the only reason I mentioned it.) |
I used shfmt (https://github.com/mvdan/sh) and i also checked and there is a new line in the file locally unsure why git is not picking this up, sorry |
Thank you for your work on the fuzzer! LGTM. |
This pull request is needed so that i can update the oss-fuzz builder located at https://github.com/fuzzy-boiii23a/oss-fuzz/tree/master/projects/nokogiri which makes it much more seamless to add new fuzz targets in this nokogiri repo which will automatically be pulled by oss-fuzz and fuzzed instead of maintaining the fuzzer in the oss-fuzz repo.