Utilized min function for verbosity line 25#46
Utilized min function for verbosity line 25#46fisayoadabs wants to merge 3 commits intogrowlf:mainfrom
Conversation
|
Pull requests should contain useful information about the reasoning behind the changes for the repository maintainer to enable them to prioritize PRs. This is part of the collaborative workflow. |
|
The PR should have been made to the ensf400-ws24 branch, not to main. |
growlf
left a comment
growlf
left a comment
There was a problem hiding this comment.
replace the lines in the docker-compose.yml file - they are needed for using this image as a development tool.
There was a problem hiding this comment.
This change will actually remove useful code. Please look at the other service definitions in this file.
There was a problem hiding this comment.
This change is from an existing branch and is not needed here - once that branch is merged.
| - ${HOME}/.gitconfig:/home/ubuntu/.gitconfig | ||
| - ${HOME}/.ssh:/home/ubuntu/.ssh | ||
| - ${PWD}:/app |
There was a problem hiding this comment.
This is for local credentials and allows development entirely within the safety of the container.
3 participants
No description provided.