Skip to content

Commit

Permalink
make docker build working with after a fresh checkout
Browse files Browse the repository at this point in the history
  • Loading branch information
Zoltan committed Mar 7, 2019
1 parent 17267bd commit 4f0dbd4
Show file tree
Hide file tree
Showing 9 changed files with 530 additions and 581 deletions.
4 changes: 4 additions & 0 deletions .dockerignore
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Dockerfile
node_modules

This comment has been minimized.

Copy link
@aslafy-z

aslafy-z Mar 7, 2019

These lines might be replaced by

**/node_modules
*/node_modules
*/*/node_modules
*/*/*/node_modules
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,4 @@ dist
out
.DS_Store
release
yarn-error.log
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ RUN npm install -g yarn
# directly which should be faster.
WORKDIR /src
COPY . .
RUN yarn
RUN yarn --forzen-lockfile

This comment has been minimized.

Copy link
@aslafy-z

aslafy-z Mar 7, 2019

Typo here: frozen

RUN yarn task build:server:binary

# We deploy with ubuntu so that devs have a familiar environemnt.
Expand Down
2 changes: 1 addition & 1 deletion build/tasks.ts
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ const ensureCloned = register("vscode:clone", async (runner) => {
}

runner.cwd = vscodePath;
const checkout = await runner.execute("git", ["checkout", "tags/1.31.1"]);
const checkout = await runner.execute("git", ["checkout", "tags/1.31.1"]); //TODO: this tag should come from a parameter
if (checkout.exitCode !== 0) {
throw new Error(`Failed to checkout: ${checkout.stderr}`);
}
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
"description": "Run VS Code remotely.",
"scripts": {
"build:rules": "cd ./rules && tsc -p .",
"packages:install": "cd ./packages && yarn",
"postinstall": "npm-run-all --parallel packages:install build:rules",
"packages:install": "cd ./packages && yarn --frozen-lockfile",
"postinstall": "npm-run-all packages:install build:rules",
"start": "cd ./packages/server && yarn start",
"task": "ts-node -r tsconfig-paths/register build/tasks.ts",
"test": "cd ./packages && yarn test"
Expand Down
508 changes: 216 additions & 292 deletions packages/app/browser/yarn.lock

Large diffs are not rendered by default.

10 changes: 5 additions & 5 deletions packages/app/common/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,12 @@
"name": "@coder/app-common",
"main": "src/app.ts",
"dependencies": {
"material-components-web": "^0.44.0",
"react": "^16.8.1",
"react-dom": "^16.8.1"
"material-components-web": "^0.44.1",
"react": "^16.8.4",
"react-dom": "^16.8.4"
},
"devDependencies": {
"@types/react": "^16.8.2",
"@types/react-dom": "^16.8.0"
"@types/react": "^16.8.7",
"@types/react-dom": "^16.8.2"
}
}
576 changes: 298 additions & 278 deletions packages/app/common/yarn.lock

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions scripts/install-packages.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const doInstall = (pkg: string, path: string): Promise<void> => {
logger.info(`Installing "${pkg}" dependencies...`);

return new Promise((resolve): void => {
exec("yarn --network-concurrency 1", {
exec("yarn --frozen-lockfile --network-concurrency 1", {
cwd: path,
maxBuffer: 1024 * 1024 * 10,
}, (error, stdout, stderr) => {
Expand Down Expand Up @@ -41,7 +41,7 @@ const handlePackages = async (dir: string): Promise<void> => {
const pkgDir = join(dir, pkg);
const pkgJsonPath = join(pkgDir, "package.json");
if (existsSync(pkgJsonPath)) {
const ip = doInstall(pkg, pkgDir);
const ip = await doInstall(pkg, pkgDir);
if (os.platform() === "win32") {
await ip;
}
Expand Down

0 comments on commit 4f0dbd4

Please sign in to comment.