Skip to content

Commit

Permalink
Merge pull request #94 from stuartleeks/sl/rm-on-run
Browse files Browse the repository at this point in the history
Add --rm when running the container
  • Loading branch information
stuartleeks authored Jul 17, 2021
2 parents 23289aa + 0954028 commit 9b2e8cc
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion azdo-task/DevContainerBuildRun/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion azdo-task/DevContainerBuildRun/dist/index.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion common/src/docker.ts
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ export async function runContainer(
const workdir = path.join(workspaceFolder, subFolder)
const remoteUser = config.getRemoteUser(devcontainerConfig)

const args = ['run']
const args = ['run', '--rm']
args.push(
'--mount',
`type=bind,src=${checkoutPathAbsolute},dst=${workspaceFolder}`
Expand Down
2 changes: 1 addition & 1 deletion common_lib/docker.js
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ function runContainer(exec, imageName, checkoutPath, subFolder, command, envs, m
const workspaceFolder = config.getWorkspaceFolder(devcontainerConfig, checkoutPathAbsolute);
const workdir = path_1.default.join(workspaceFolder, subFolder);
const remoteUser = config.getRemoteUser(devcontainerConfig);
const args = ['run'];
const args = ['run', '--rm'];
args.push('--mount', `type=bind,src=${checkoutPathAbsolute},dst=${workspaceFolder}`);
if (devcontainerConfig.mounts) {
devcontainerConfig.mounts
Expand Down
2 changes: 1 addition & 1 deletion github-action/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion github-action/dist/index.js.map

Large diffs are not rendered by default.

0 comments on commit 9b2e8cc

Please sign in to comment.