Skip to content

Conversation

@DavidKorczynski
Copy link
Contributor

Add CIFuzz workflow action to have fuzzers build and run on each PR.

This is a service offered by OSS-Fuzz where ipfs/go-datastore already runs (https://github.com/google/oss-fuzz/tree/master/projects/ipfs). CIFuzz can help detect regressions and catch fuzzing build issues early, and has a variety of features (see the URL above). In the current PR the fuzzers gets build on a pull request and will run for 300 seconds.

@welcome
Copy link

welcome bot commented Nov 20, 2022

Thank you for submitting this PR!
A maintainer will be here shortly to review it.
We are super grateful, but we are also overloaded! Help us by making sure that:

  • The context for this PR is clear, with relevant discussion, decisions
    and stakeholders linked/mentioned.

  • Your contribution itself is clear (code comments, self-review for the
    rest) and in its best form. Follow the code contribution
    guidelines

    if they apply.

Getting other community members to do a review would be great help too on complex PRs (you can ask in the chats/forums). If you are unsure about something, just leave us a comment.
Next steps:

  • A maintainer will triage and assign priority to this PR, commenting on
    any missing things and potentially assigning a reviewer for high
    priority items.

  • The PR gets reviews, discussed and approvals as needed.

  • The PR is merged by maintainers when it has been approved and comments addressed.

We currently aim to provide initial feedback/triaging within two business days. Please keep an eye on any labelling actions, as these will indicate priorities and status of your contribution.
We are very grateful for your contribution!

@aschmahmann
Copy link
Contributor

aschmahmann commented Mar 6, 2023

@ipfs/ipdx should this be here, or in Unified CI (https://github.com/protocol/.github)?

@galargh
Copy link
Contributor

galargh commented Mar 6, 2023

If it's a completely new workflow, I'd go with trying it out on a repo first and then proposing it for Unified CI.

@guseggert guseggert added P2 Medium: Good to have, but can wait until someone steps up need/maintainer-input Needs input from the current maintainer(s) labels Apr 3, 2023
@aschmahmann aschmahmann merged commit 0cd67a2 into ipfs:master May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

need/maintainer-input Needs input from the current maintainer(s) P2 Medium: Good to have, but can wait until someone steps up

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants