Skip to content

✨ Support OneFuzz in fuzzing checks#2141

Merged
naveensrinivasan merged 4 commits intoossf:mainfrom
balteravishay:avbalter/add-onefuzz-detection
Aug 12, 2022
Merged

✨ Support OneFuzz in fuzzing checks#2141
naveensrinivasan merged 4 commits intoossf:mainfrom
balteravishay:avbalter/add-onefuzz-detection

Conversation

@balteravishay
Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Add support for detecting OneFuzz in fuzzing checks.

What is the current behavior?

The fuzzing check of scorecard checks for OSS-Fuzz, ClusterFuzzLite and in a repo user-defined golang fuzz functions in the repo.

What is the new behavior (if this is a feature change)?**

Check for the existence of a OneFuzz detection file in the repo and give the check a full score (10/10) if such file is found.

  • [v] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Special notes for your reviewer

Does this PR introduce a user-facing change?

Yes

For user-facing changes, please add a concise, human-readable release note to
the release-note

(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

Users now can check for OneFuzz integration in repo as a part of the fuzzing check.

@balteravishay balteravishay temporarily deployed to integration-test August 12, 2022 08:43 Inactive
@balteravishay balteravishay changed the title :sparkles Support OneFuzz in fuzzing checks ✨ Support OneFuzz in fuzzing checks Aug 12, 2022
@balteravishay balteravishay changed the title ✨ Support OneFuzz in fuzzing checks :sparkles: Support OneFuzz in fuzzing checks Aug 12, 2022
@balteravishay balteravishay changed the title :sparkles: Support OneFuzz in fuzzing checks (:sparkles:) Support OneFuzz in fuzzing checks Aug 12, 2022
@balteravishay balteravishay changed the title (:sparkles:) Support OneFuzz in fuzzing checks ✨ Support OneFuzz in fuzzing checks Aug 12, 2022
@github-actions
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 12, 2022

Codecov Report

Merging #2141 (1564898) into main (0d9b7a1) will increase coverage by 2.44%.
The diff coverage is 48.14%.

@@            Coverage Diff             @@
##             main    #2141      +/-   ##
==========================================
+ Coverage   42.63%   45.07%   +2.44%     
==========================================
  Files          94       94              
  Lines        7777     7804      +27     
==========================================
+ Hits         3316     3518     +202     
+ Misses       4203     4021     -182     
- Partials      258      265       +7     

Copy link
Copy Markdown
Contributor

@azeemshaikh38 azeemshaikh38 left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM. @laurentsimon for a second look.

@azeemshaikh38 azeemshaikh38 temporarily deployed to integration-test August 12, 2022 14:05 Inactive
@github-actions
Copy link
Copy Markdown

@naveensrinivasan naveensrinivasan merged commit abcd409 into ossf:main Aug 12, 2022
@laurentsimon
Copy link
Copy Markdown
Contributor

Thank you @balteravishay !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants