Skip to content

Conversation

thecharlieblake
Copy link
Contributor

No description provided.

@thecharlieblake thecharlieblake marked this pull request as draft November 5, 2024 09:28
@thecharlieblake thecharlieblake marked this pull request as ready for review November 5, 2024 09:28
Copy link
Collaborator

@DouglasOrr DouglasOrr left a comment

Choose a reason for hiding this comment

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

LGTM; thanks!

env: { GITHUB_TOKEN: "${{ github.token }}" }
with:
build_dir: docs/_build/html
build_dir: docker run --rm -v $(pwd):/workspace unit-scaling-dev:latest docs/_build/html
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this right - I thought it looked like a dir not a command?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's completely mad! No idea what's going on there. Removed

# Creates basic .zshrc
RUN sudo cp /etc/zsh/newuser.zshrc.recommended /home/$USERNAME/.zshrc

CMD ["/bin/zsh"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why zsh, out of interest? Just personal preference? (No objection, just curious.)

@thecharlieblake thecharlieblake merged commit bcc3d5e into main Nov 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants