sausheong/invaders
1
0
Fork 0
Code Issues 4 Pull requests 3 Projects Releases Packages Wiki Activity Actions

Dockerfile #6

Open
jangaraj wants to merge 2 commits from unknown repository into master
pull from: unknown repository
merge into: sausheong:master
Conversation 2 Commits 2 Files changed 1 +14
jangaraj commented 2018-02-02 19:17:06 -05:00 (Migrated from github.com)
Copy link

Dockerfile for those, who like golang builds in the Docker container. It produces minimalistic Docker image (2.22MB).
Docker build: docker build -t invaders .
Unfortunately, it doesn't work in my Putty, so I was not able to test it in my console.
Public Docker image for testing: docker run --rm -ti jangaraj/invaders

Dockerfile for those, who like golang builds in the Docker container. It produces minimalistic Docker image (2.22MB). Docker build: `docker build -t invaders .` Unfortunately, it doesn't work in my Putty, so I was not able to test it in my console. Public Docker image for testing: `docker run --rm -ti jangaraj/invaders`
vosmith commented 2018-02-03 09:56:05 -05:00 (Migrated from github.com)
Copy link

Hey @jangaraj I pulled your branch to try your docker file against my pixterm PR. The only change I had to make was adding a WORKDIR / somehere after FROM scratch. This ensures that you are running invaders from the same directory that the imgs/ are, which is necessary for now.

Hey @jangaraj I pulled your branch to try your docker file against my [pixterm PR](https://github.com/sausheong/invaders/pull/7). The only change I had to make was adding a `WORKDIR /` somehere after `FROM scratch`. This ensures that you are running `invaders` from the same directory that the `imgs/` are, which is necessary for now.
jangaraj commented 2018-02-03 09:59:49 -05:00 (Migrated from github.com)
Copy link

@vosmith thx, fixed

@vosmith thx, fixed
This pull request is broken due to missing fork information.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin unknown repository:unknown repository
git switch unknown repository

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff unknown repository
git switch unknown repository
git rebase master
git switch master
git merge --ff-only unknown repository
git switch unknown repository
git rebase master
git switch master
git merge --no-ff unknown repository
git switch master
git merge --squash unknown repository
git switch master
git merge --ff-only unknown repository
git switch master
git merge unknown repository
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
sausheong/invaders!6
No description provided.