Skip to content

feat: add environment.shells for darwin - #648

Merged
theutz merged 1 commit into
denful:mainfrom
phucisstupid:patch-2
Aug 7, 2026
Merged

feat: add environment.shells for darwin#648
theutz merged 1 commit into
denful:mainfrom
phucisstupid:patch-2

Conversation

@phucisstupid

Copy link
Copy Markdown
Contributor

on macos we must add the shell to environment.shells and change the default shell using chsh -s

@phucisstupid
phucisstupid requested review from sini and vic as code owners August 4, 2026 14:05

@sini sini left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sini sini added the allow-ci allow all CI integration tests label Aug 4, 2026
@sini

sini commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

@phucisstupid I'll merge this after you fix the linting errors; you just need to run nix develop -c just fmt

on macos we must add the shell to `environment.shells` and change the default shell using `chsh -s`

Signed-off-by: phuclee <125681538+phucisstupid@users.noreply.github.com>
@phucisstupid

Copy link
Copy Markdown
Contributor Author

@sini done

@theutz
theutz merged commit 49c7002 into denful:main Aug 7, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow-ci allow all CI integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants