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

CompFuzzCI integration (no forks) #5697

Merged
merged 5 commits into from
Aug 19, 2024
Merged

CompFuzzCI integration (no forks) #5697

merged 5 commits into from
Aug 19, 2024

Conversation

stefan-aws
Copy link
Collaborator

Description

How has this been tested?

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

branches:
- master
types: [opened, closed, synchronize, reopened]
issues:
Copy link
Member

Choose a reason for hiding this comment

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

Why is issues here? Why should this trigger when someone makes an issue?

Copy link
Contributor

@kbuaaaaaa kbuaaaaaa Aug 15, 2024

Choose a reason for hiding this comment

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

It's for keeping track of active bugs in dafny. If the fuzzer finds the bug that has already been reported in some issue, it can ignore that bug.

Should I put issue processing in a separate workflow?

@@ -0,0 +1,91 @@
name: CompFuzzCI
Copy link
Member

Choose a reason for hiding this comment

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

Could you add a comment to the top of this file that explains what it does?

Copy link
Contributor

Choose a reason for hiding this comment

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

Good idea. I'll get that done. Thanks!

pull_request:
branches:
- master
types: [opened, closed, synchronize, reopened]
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 the types list for?

Copy link
Contributor

Choose a reason for hiding this comment

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

I want to trigger workflow on pull request getting closed as well, this does not happen by default, hence the list.
By default pull_request event trigger on pull request being opened, synced, reopened.

@keyboardDrummer keyboardDrummer merged commit d21e1b9 into master Aug 19, 2024
22 checks passed
@keyboardDrummer keyboardDrummer deleted the no-fork-fuzzing branch August 19, 2024 09:40
dnezam pushed a commit to dnezam/dafny that referenced this pull request Sep 21, 2024
### Description
<!-- Is this a user-visible change? Remember to update RELEASE_NOTES.md
-->
<!-- Is this a bug fix for an issue visible in the latest release?
Mention this in the PR details and ensure a patch release is considered
-->

### How has this been tested?
<!-- Tests can be added to
`Source/IntegrationTests/TestFiles/LitTests/LitTest/` or to
`Source/*.Test/…` and run with `dotnet test` -->

<small>By submitting this pull request, I confirm that my contribution
is made under the terms of the [MIT
license](https://github.com/dafny-lang/dafny/blob/master/LICENSE.txt).</small>
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