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

ci: Adjust fuzzing config #559

Merged
merged 1 commit into from
Sep 30, 2020
Merged

ci: Adjust fuzzing config #559

merged 1 commit into from
Sep 30, 2020

Conversation

chfast
Copy link
Collaborator

@chfast chfast commented Sep 29, 2020

No description provided.

@codecov
Copy link

codecov bot commented Sep 29, 2020

Codecov Report

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

@@           Coverage Diff           @@
##           master     #559   +/-   ##
=======================================
  Coverage   98.64%   98.64%           
=======================================
  Files          59       59           
  Lines        8948     8948           
=======================================
  Hits         8827     8827           
  Misses        121      121           

@chfast chfast force-pushed the ci_fuzzing branch 5 times, most recently from 1fcc7b8 to 7b1a20a Compare September 30, 2020 06:57
@chfast chfast marked this pull request as ready for review September 30, 2020 06:58
@chfast chfast requested review from axic and gumb0 September 30, 2020 06:58
@@ -454,22 +454,37 @@ jobs:
- build:
build_type: RelWithDebInfo
cmake_options: -DFIZZY_FUZZING=ON -DENABLE_ASSERTIONS=ON
target: fizzy-fuzz-parser
Copy link
Member

Choose a reason for hiding this comment

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

What is this doing?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It only builds fizzy-fuzz-parser instead of building everything, so the build takes only ~15 seconds. We don't run tests in this job (although we could, they would be similar to sanitizers-clang job).

Copy link
Member

Choose a reason for hiding this comment

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

It means build target, e.g. make fizzy-fuzz-parser?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes.

@axic axic merged commit fc76928 into master Sep 30, 2020
@axic axic deleted the ci_fuzzing branch September 30, 2020 14:16
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.

2 participants