Skip to content

Commit

Permalink
fix asset size check
Browse files Browse the repository at this point in the history
  • Loading branch information
runspired committed Jul 23, 2022
1 parent 47789ab commit bfaa859
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/asset-size-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
- name: Install dependencies for master
run: yarn install
- name: Build Production master
run: EMBER_DATA_FULL_COMPAT=true yarn workspace ember-data ember build -e production --output-path dists/control
run: IS_ASSET_SIZE_CHECK=true EMBER_DATA_FULL_COMPAT=true yarn workspace ember-data ember build -e production --output-path dists/control
- name: Build Production master (no rollup)
run: EMBER_DATA_FULL_COMPAT=true EMBER_DATA_ROLLUP_PRIVATE=false yarn workspace ember-data ember build -e production --output-path dists/control-no-rollup
- name: Checkout ${{github.ref}}
Expand All @@ -46,7 +46,7 @@ jobs:
- name: Install dependencies for ${{github.ref}}
run: yarn install
- name: Build Production ${{github.ref}}
run: EMBER_DATA_FULL_COMPAT=true yarn workspace ember-data ember build -e production --output-path dists/experiment
run: IS_ASSET_SIZE_CHECK=true EMBER_DATA_FULL_COMPAT=true yarn workspace ember-data ember build -e production --output-path dists/experiment
- name: Build Production ${{github.ref}} (no rollup)
run: EMBER_DATA_FULL_COMPAT=true EMBER_DATA_ROLLUP_PRIVATE=false yarn workspace ember-data ember build -e production --output-path dists/experiment-no-rollup
- name: Analyze Master Assets
Expand Down
4 changes: 2 additions & 2 deletions packages/-ember-data/ember-cli-build.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,15 @@ module.exports = function (defaults) {
defaults: true,
arguments: true,
keep_fargs: false,
toplevel: true,
toplevel: process.env.IS_ASSET_SIZE_CHECK ? false : true,
unsafe: true,
unsafe_comps: true,
unsafe_math: true,
unsafe_symbols: true,
unsafe_proto: true,
unsafe_undefined: true,
},
toplevel: true,
toplevel: process.env.IS_ASSET_SIZE_CHECK ? false : true,
sourceMap: false,
ecma: 2016,
},
Expand Down

0 comments on commit bfaa859

Please sign in to comment.