Running chsh at the end of bootstrap is not needed in CI. It's always going to fail and it doesn't provide us any context.

Azure cli is required for one of the functions in the docker wrapper so it should be required by the bootstrap script.

Any installed commands being used in functions that conflict with installed command names should be prefixed with command.

Some patterns were applied blindly to some functions that didn't make sense. For example, the --wraps or -w command is applied in a few places where it shouldn't be. Fix all function definitions.

In the spirit of matching "ShellCheck"

Validate utilities like git are using the homebrew installation, not the built in version

https://apple.stackexchange.com/a/93179

I lost all of my prior neovim configs in the transition to this new setup. Truthfully, they were a bit over the top anyway so this is a good restarting point.

Automatically lint bash scripts to verify they were written properly before merging into main

Use fish's no-execute flag to check for issues.

Example:

cat .config/fish/functions/git.fish | fish -n

This isn't the best check in the world, but it's better than nothing.

I'm unable to use the smoke test as a status check without a job name

https://stackoverflow.com/a/68613319