Package building isn't part of our CI.

[mkcor]

Whenever we change JS code, we rebuild the package by running

npm run build:all

which results in dash_bio/bundle.js. If this breaks, we have a broken package. Therefore, it is important to make this command part of CI (it's currently missing in our CI config).

Not only do we want to double check builds that were successful locally (maybe our dev environments aren't fully up-to-date and/or we don't run both Python 3.6 and Python 3.7), but we also want PRs from the Renovate Bot (dependency updates, such #299 and #284) to go through the full pipeline.

[shammamah-zz]

@mkcor Shouldn't it go in the node part of the config?

dash-bio/.circleci/config.yml

Line 27 in fc25489

[mkcor]

Indeed, it's a node job:

dash-bio/.circleci/config.yml

Line 94 in fc25489

jobs:

All I meant was that I went through all the runs and none included package building, i.e., there are no more runs after:

dash-bio/.circleci/config.yml

Line 80 in fc25489