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.

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

fix: install zsh before changing shell #224

Merged
merged 2 commits into from
Sep 6, 2023

Conversation

spameier
Copy link
Contributor

@spameier spameier commented Sep 5, 2023

Description

Install zsh before changing the shell to it.

Related issues

fixes #223

@QU35T-code
Copy link
Member

zsh is installed via the apt list in the line below. In my opinion you just have to move the chsh below the fapt list

@ShutdownRepo
Copy link
Member

Running the pipeline since we made various changes in them lately and I'd like to see how they run, but in my opinion we can merge this PR without

@ShutdownRepo ShutdownRepo added ready for merge in the waitlist for merge, requires preliminary steps bugfix It's a bird, it's a plane labels Sep 5, 2023
@ShutdownRepo ShutdownRepo linked an issue Sep 5, 2023 that may be closed by this pull request
@QU35T-code
Copy link
Member

Agree to merge now

@ShutdownRepo ShutdownRepo removed the ready for merge in the waitlist for merge, requires preliminary steps label Sep 6, 2023
@ShutdownRepo ShutdownRepo merged commit 8eec3be into ThePorgs:dev Sep 6, 2023
38 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix It's a bird, it's a plane
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] zsh missing from base image
3 participants