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

Fix the fuzz bug #11

Merged
merged 3 commits into from
Aug 22, 2024
Merged

Fix the fuzz bug #11

merged 3 commits into from
Aug 22, 2024

Commits on Aug 21, 2024

  1. Ignore dynamic paths with shellcheck 

    We source a bunch of dynamic paths, `shellcheck` cannot handle dynamic
paths - configure `shellcheck` to ignore the warnings.
    @tcharding
    tcharding committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    274bdbb View commit details
    Browse the repository at this point in the history

  2. Use say_err 

    We have an error function, use it.

This clears a shellcheck warning (unseen locally by myself) and puts the
error message below the ouput from `cargo tree` which is probably better
if its long.
    @tcharding
    tcharding committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    97fba0c View commit details
    Browse the repository at this point in the history

  3. Stop treating fuzz as a special case 

    We would like to run the build and tests for all crates in the
workspace. In order to do so clear the env vars from the last
loop (setting to the empty string) and check against the empty string
when running specific `cargo` incantations.
    @tcharding
    tcharding committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    6646692 View commit details
    Browse the repository at this point in the history