justinmoon / junction

UI For Using Hardware Wallets With Bitcoin Core
MIT License
60 stars 13 forks source link

Failed to compile. #45

Closed vdo closed 3 years ago

vdo commented 4 years ago
Failed to compile.

/home/vdo/src/junction/node_modules/jest-diff/build/diffLines.d.ts
TypeScript error in /home/vdo/src/junction/node_modules/jest-diff/build/diffLines.d.ts(8,13):
'=' expected.  TS1005

     6 |  */
     7 | import { Diff } from './cleanupSemantic';
  >  8 | import type { DiffOptions } from './types';
       |             ^
     9 | export declare const diffLinesUnified: (aLines: Array<string>, bLines: Array<string>, options?: DiffOptions | undefined) => string;
    10 | export declare const diffLinesUnified2: (aLinesDisplay: Array<string>, bLinesDisplay: Array<string>, aLinesCompare: Array<string>, bLinesCompare: Array<string>, options?: DiffOptions | undefined) => string;
    11 | export declare const diffLinesRaw: (aLines: Array<string>, bLines: Array<string>) => Array<Diff>;
wbobeirne commented 4 years ago

Hey @vdo, import type is some newer TypeScript syntax that was added a decent while after Junction's latest commit. Make sure you install packages using yarn (not npm) to make sure the correct dependency versions are respected as defined in yarn.lock. Otherwise it may pull in newer dependencies with breaking changes, such as the one you found here.

Make sure you rm package-lock.json && rm -rf node_modules before installing to make sure you clear out the old packages from the npm install.