[fpringle]

Overview

scripts/hooks/pre-push.bash and other bash scripts had a #!/bin/bash shebang instead of #!/usr/bin/env bash. /bin/bash is less portable, for example it doesn't exist on NixOS.

Implementation approach and notes

Doesn't change the Haskell codebase at all.

Interesting/controversial decisions

None

Test coverage

• If you only tested by hand, because that's all that's practical to do for this change, mention that. Include screenshots.

Before:

After:

Loose ends

None.

Final checklist

• Choose your PR title well: Your pull request title is what's used to create release notes, so please make it descriptive of the change itself, which may be different from the initial motivation to make the change.
• Update your PR description if the specifics of the PR have changed over time.
• Include transcripts or screenshots that demonstrate the changed behavior.
• If you changed .cabal files, make sure the package.yaml files are up-to-date instead.

[github-actions]

🤖 The merge queue's full run failed, which removed this pull request from the queue: https://github.com/unisonweb/unison/actions/runs/42817907157

[pchiusano]

Full CI failed due to flaky windows test, which I'll fix separately, will try again.