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.

Sign up for GitHub

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

Jump to bottom

Reenable CI fuzzing of pull requests #1596

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Reenable CI fuzzing of pull requests #1596

merged 3 commits into from
Sep 20, 2024

Commits on Sep 15, 2024

  1. Revert "Looks like fuzzing is failing permanently again" 

    This reverts commit 3d90ab0.
    @EliahKagan
    EliahKagan committed Sep 15, 2024
    Configuration menu
    Copy the full SHA
    25893ed View commit details
    Browse the repository at this point in the history

  2. Try manully specifying old serde{,_derive,_json} in Cargo.toml 

    As before GitoxideLabs#1536.

I'm not sure CI fuzzing actually uses locked dependencies, though.
    @EliahKagan
    EliahKagan committed Sep 15, 2024
    Configuration menu
    Copy the full SHA
    aafa8b1 View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2024

  1. Revert "Try manully specifying old serde{,_derive,_json} in `Cargo.to… 

    …ml`"

This reverts commit aafa8b1, since
fuzzing doesn't use locked dependencies, so that's not a fix. See:

- GitoxideLabs#1596 (comment)
- google/oss-fuzz#12512, which does fix it
    @EliahKagan
    EliahKagan committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    39cab61 View commit details
    Browse the repository at this point in the history