Merge pull request #220 from kategengler/kg-fix-build

Author: kategengler

.github/workflows/build.yml

@@ -26,14 +26,14 @@ jobs:
           git config --global user.name "Tomster"
           git config --global user.email "tomster@emberjs.com"
       - name: Set up Volta
-        uses: rwjblue/setup-volta@v1
+        uses: volta-cli/action@v4
       - name: Set up build cache (yarn)
-        uses: actions/cache@preview
+        uses: actions/cache@v3
         with:
           path: ~/.cache/yarn
           key: ${{ runner.os }}-yarn
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
       - name: Install dependencies (apt-get)
         run: |
           sudo apt-get update -y
@@ -69,19 +69,19 @@ jobs:
         working-directory: dist/code/super-rentals
         run: git clean -dfX
       - name: Upload artifacts (assets)
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         if: always()
         with:
           name: assets (${{ matrix.channel }})
           path: dist/assets
       - name: Upload artifacts (markdown)
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         if: always()
         with:
           name: markdown (${{ matrix.channel }})
           path: dist/markdown
       - name: Upload artifacts (code)
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         if: always()
         with:
           name: code (${{ matrix.channel }})
@@ -142,12 +142,12 @@ jobs:
           sudo apt-get update -y
           sudo apt-get install -y zipcmp advancecomp optipng perceptualdiff
       - name: Download artifacts (assets)
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: assets (release)
           path: assets
       - name: Download artifacts (markdown)
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: markdown (release)
           path: markdown
@@ -240,7 +240,7 @@ jobs:
 
           git add public/images
       - name: Upload artifacts (perceptualdiff)
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         if: always()
         with:
           name: perceptualdiff (release)
@@ -305,7 +305,7 @@ jobs:
         with:
           ssh-private-key: ${{ secrets.SUPER_RENTALS_DEPLOY_KEY }}
       - name: Download artifacts
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: code (release)
           path: .
@@ -364,7 +364,7 @@ jobs:
         with:
           ssh-private-key: ${{ secrets.SUPER_RENTALS_DEPLOY_KEY }}
       - name: Download artifacts
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: code (release)
           path: output

package.json

@@ -8,23 +8,22 @@
   "private": true,
   "devDependencies": {
     "@actions/core": "^1.2.6",
-    "@types/glob": "^7.1.1",
-    "@types/image-size": "^0.7.0",
+    "@types/glob": "^8.0.0",
+    "@types/image-size": "^0.8.0",
     "@types/mdast": "^3.0.2",
-    "@types/mkdirp": "^0.5.2",
+    "@types/mkdirp": "^1.0.2",
     "@types/ncp": "^2.0.1",
-    "@types/node": "^12.7.2",
-    "@types/puppeteer": "^2.0.0",
+    "@types/node": "^18.11.18",
     "@types/unist": "^2.0.3",
-    "@types/vfile": "^3.0.2",
+    "@types/vfile": "^4.0.0",
     "chalk": "^2.4.2",
     "ember-cli": "*",
     "ember-dictionary": "^0.2.3",
-    "image-size": "^0.7.4",
-    "mkdirp": "^0.5.1",
+    "image-size": "^1.0.2",
+    "mkdirp": "^1.0.4",
     "ncp": "^2.0.0",
     "node-glob": "^1.2.0",
-    "puppeteer": "~2.1.1",
+    "puppeteer": "~19.5.2",
     "rehype-stringify": "^6.0.0",
     "remark": "^11.0.1",
     "remark-cli": "^7.0.0",
@@ -46,9 +45,9 @@
     "retext-syntax-urls": "^1.0.2",
     "shx": "^0.3.2",
     "ts-std": "^0.7.0",
-    "tslint": "^5.18.0",
+    "tslint": "^6.1.3",
     "tslint-config-prettier": "^1.18.0",
-    "typescript": "^3.5.3",
+    "typescript": "^4.9.4",
     "unified": "^8.4.0"
   },
   "scripts": {

src/bin/generate.ts

@@ -5,7 +5,7 @@ import {
   writeFile as _writeFile
 } from 'fs';
 import _glob from 'glob';
-import _mkdirp from 'mkdirp';
+import mkdirp from 'mkdirp';
 import { ncp as _ncp } from 'ncp';
 import { basename, dirname, join, relative, sep } from 'path';
 import frontmatter from 'remark-frontmatter';
@@ -21,7 +21,6 @@ import { doNotEdit, retinaImages, runCodeBlocks, todoLinks, zoeySays } from '../
 const glob = promisify(_glob);
 const readFile = promisify(_readFile);
 const writeFile = promisify(_writeFile);
-const mkdirp = promisify(_mkdirp);
 const ncp = promisify(_ncp);
 
 // 01-orientation.md -> orientation.md
@@ -55,8 +54,8 @@ async function main() {
   let codeDir = join(project, 'dist', 'code');
 
   await ncp(join(project, 'src', 'assets'), assetsDir);
-  await mkdirp(outDir);
-  await mkdirp(codeDir);
+  await mkdirp(outDir, {});
+  await mkdirp(codeDir, {});
 
   let pattern = process.argv[2] || join('src', 'markdown', '**', '*.md');
 
@@ -79,7 +78,7 @@ async function main() {
       let dir = unprefix(relative(srcDir, dirname(inputPath)));
       let name = unprefix(basename(inputPath));
 
-      await mkdirp(join(outDir, dir));
+      await mkdirp(join(outDir, dir), {});
 
       let outputPath = join(outDir, dir, name);
 

src/lib/plugins/retina-images/walker.ts

@@ -2,6 +2,7 @@ import { exists as _exists } from 'fs';
 import _imageSize from 'image-size';
 import { HTML, Image } from 'mdast';
 import { basename, resolve } from 'path';
+import { expect } from 'ts-std';
 import { promisify } from 'util';
 import BaseWalker from '../../walker';
 import Options from './options';
@@ -38,9 +39,14 @@ function attr(v: unknown): string {
 
 
 async function toImgTag(node: Image, options: Options): Promise<HTML> {
-  let size = await imageSize(pathFor(node.url, options));
-  let width = Math.floor(size.width / 2);
-  let height = Math.floor(size.height / 2);
+  let sizeOfImage = await imageSize(pathFor(node.url, options));
+
+  let size = expect(sizeOfImage, 'size should be present');
+  let width = expect(size.width, 'width should be present');
+  let height = expect(size.height, 'height should be present');
+
+  let widthAttr = Math.floor(width / 2);
+  let heightAttr = Math.floor(height / 2);
 
   let attrs = [];
 
@@ -54,8 +60,8 @@ async function toImgTag(node: Image, options: Options): Promise<HTML> {
     attrs.push(`title=${attr(node.title)}`);
   }
 
-  attrs.push(`width=${attr(width)}`);
-  attrs.push(`height=${attr(height)}`);
+  attrs.push(`width=${attr(widthAttr)}`);
+  attrs.push(`height=${attr(heightAttr)}`);
 
   return {
     type: 'html',

src/lib/plugins/run-code-blocks/directives/file/copy.ts

@@ -1,6 +1,6 @@
 import { lstat as _lstat, readFile as _readFile } from 'fs';
 import { Code } from 'mdast';
-import _mkdirp from 'mkdirp';
+import mkdirp from 'mkdirp';
 import { ncp as _ncp } from 'ncp';
 import { basename, dirname, join } from 'path';
 import { Option } from 'ts-std';
@@ -9,7 +9,6 @@ import Options from '../../options';
 import parseArgs, { ToBool, optional, required } from '../../parse-args';
 
 const lstat = promisify(_lstat);
-const mkdirp = promisify(_mkdirp);
 const ncp = promisify(_ncp);
 const readFile = promisify(_readFile);
 
@@ -53,7 +52,7 @@ export default async function copyFile(node: Code, options: Options): Promise<Op
     throw new Error(`\`${src}\` is neither a regular file or a directory`);
   }
 
-  await mkdirp(destDir);
+  await mkdirp(destDir, {});
   await ncp(src, destPath);
 
   if (args.hidden) {

src/lib/plugins/run-code-blocks/directives/file/create.ts

@@ -1,13 +1,12 @@
 import { writeFile as _writeFile } from 'fs';
 import { Code } from 'mdast';
-import _mkdirp from 'mkdirp';
+import mkdirp from 'mkdirp';
 import { basename, dirname, join } from 'path';
 import { Option } from 'ts-std';
 import { promisify } from 'util';
 import Options from '../../options';
 import parseArgs, { ToBool, optional, required } from '../../parse-args';
 
-const mkdirp = promisify(_mkdirp);
 const writeFile = promisify(_writeFile);
 
 interface Args {
@@ -37,7 +36,7 @@ export default async function createFile(node: Code, options: Options): Promise<
 
   dir = join(dir, dirname(args.filename));
 
-  await mkdirp(dir);
+  await mkdirp(dir, {});
 
   let path = join(dir, basename(args.filename));
 

src/lib/plugins/run-code-blocks/directives/pause.ts

@@ -6,7 +6,7 @@ import Options from '../options';
 async function prompt(message: string): Promise<void> {
   console.log(`\n${message}\n`);
 
-  await new Promise(resolve => {
+  await new Promise<void>(resolve => {
     let rl = readline.createInterface({
       input: process.stdin,
       output: process.stdout

src/lib/plugins/run-code-blocks/directives/screenshot.ts

@@ -1,6 +1,6 @@
 import { exec as _exec } from 'child_process';
 import { Code, Image } from 'mdast';
-import _mkdirp from 'mkdirp';
+import mkdirp from 'mkdirp';
 import { basename, extname, join, sep } from 'path';
 import { ScreenshotOptions, Viewport } from 'puppeteer';
 import { JSONObject, JSONValue, assert } from 'ts-std';
@@ -10,7 +10,6 @@ import Options from '../options';
 import parseArgs, { ToBool, optional, required } from '../parse-args';
 
 const exec = promisify(_exec);
-const mkdirp = promisify(_mkdirp);
 
 interface Args {
   filename: string;
@@ -156,7 +155,7 @@ export default async function screenshot(node: Code, options: Options, vfile: VF
     filename = `${basename(filename, '.png')}@2x.png`;
   }
 
-  await mkdirp(dir);
+  await mkdirp(dir, {});
 
   let path = join(dir, filename);
 

src/lib/plugins/run-code-blocks/parse-args.ts

@@ -27,7 +27,7 @@ export function ToBool(input: string): boolean {
   return input === 'true';
 }
 
-export type KeyTransforms = Array<KeyTransform<string | undefined, unknown>>;
+export type KeyTransforms = KeyTransform<string | undefined, unknown>[];
 
 export default function parseArgs<Args extends object>(node: Code, transforms: KeyTransforms): Args {
   let { lang, meta } = node;

src/lib/plugins/run-code-blocks/server.ts

@@ -25,7 +25,7 @@ export default class Server {
     // TODO
   }
 
-  get pid(): Option<number> {
+  get pid(): null | number | undefined {
     return this.process && this.process.pid;
   }