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

Fuzzing on forks #5762

Merged

Conversation

kbuaaaaaa
Copy link
Contributor

Description

I have modified the fuzzing workflow to handle PR from forks. Fuzzing will only be triggered on PR created by verified contributors to prevent potential security risks.
ALL WILL BE FUZZED mwahaha

This change also fix the following annoying issues:

  • Reporting bugs already caught by CI: Fuzzing will now be delayed and only run if all CI checks have passed. This is implemented outside of the workflow, the PR will not be blocked.
  • Reporting non-bugs in Rust: Added pattern matching to ignore error that comes from not yet implemented features in Dafny-to-Rust. (sorry Mikael!)

How has this been tested?

Tested on my own dummy repo

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@stefan-aws
Copy link
Collaborator

LGTM

@keyboardDrummer keyboardDrummer enabled auto-merge (squash) September 11, 2024 14:06
@keyboardDrummer keyboardDrummer merged commit 3aae84a into dafny-lang:master Sep 11, 2024
21 checks passed
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.

3 participants