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: run fuzz regression tests #6168

Merged
merged 1 commit into from
Apr 12, 2023

Conversation

morehouse
Copy link
Contributor

@morehouse morehouse commented Apr 10, 2023

There are several benefits of doing this:

  • prevent fuzz target bit rot
  • more test coverage in CI
  • greater visibility of fuzz tests, encouraging contributions to the seed corpus and tests themselves

Resolves #6083.

Changelog-None: no user-visible changes --RR

There are several benefits of doing this:
- prevent fuzz target bit rot
- more test coverage in CI
- greater visibility of fuzz tests, encouraging contributions to the
  seed corpus and tests themselves
Copy link
Collaborator

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

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

ACK 4e76a1f

I agree!

Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

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

Ack 4e76a1f

Great work! Technically this is after the 1 April "PR freeze" but @ShahanaFarooqui may choose to include it in this release.

@ShahanaFarooqui ShahanaFarooqui added this to the v23.05 milestone Apr 12, 2023
@ShahanaFarooqui ShahanaFarooqui merged commit 3c4b20e into ElementsProject:master Apr 12, 2023
28 checks passed
@morehouse morehouse deleted the fuzz_ci branch April 12, 2023 14:41
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.

Follow fuzzing best practices in CI
4 participants