Skip to content

Commit

Permalink
fix(build): add auth-client cjs build to 'postinstall'
Browse files Browse the repository at this point in the history
Because:
 - it is 'postinstall' that builds fxa-auth-client when npm/yarn install
   is called
   - the cjs build call was missing in 'postinstall'

This commit:
 - adds cjs build to 'postinstall'
  • Loading branch information
chenba committed Jul 24, 2023
1 parent ebcaec1 commit 5fbca8b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 15 deletions.
13 changes: 0 additions & 13 deletions packages/fxa-auth-client/.cjs.package.json

This file was deleted.

4 changes: 2 additions & 2 deletions packages/fxa-auth-client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@
},
"scripts": {
"lint": "eslint . --ext .ts",
"postinstall": "(tsc --build tsconfig.browser.json && tsc --build) || true",
"build": "tsc --build tsconfig.browser.json && tsc --build && tsc --build tsconfig.cjs.json && cp .cjs.package.json ./dist/server/cjs/packages/fxa-auth-client/package.json",
"postinstall": "(tsc --build tsconfig.browser.json && tsc --build && tsc --build tsconfig.cjs.json) || true",
"build": "tsc --build tsconfig.browser.json && tsc --build && tsc --build tsconfig.cjs.json",
"compile": "tsc --noEmit",
"ts-check": "tsc --noEmit",
"test": "mocha -r esbuild-register test/*",
Expand Down

0 comments on commit 5fbca8b

Please sign in to comment.