# Understanding client-side tools - Deployment

**URL:** <https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668>\
**Category:** Learn\
**Created:** [August 30, 2020, 6:08am UTC](https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668 "2020-08-30T06:08:46Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![Rafael\_Green](https://avatars.discourse-cdn.com/v4/letter/r/74df32/32.png) [@Rafael\_Green](https://discourse.mozilla.org/u/Rafael_Green)\
**Post date:** [August 30, 2020, 6:08am UTC](https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668/1 "2020-08-30T06:08:46Z")

</div>

Hi,  
I’m reading the [Deploying our app](https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Understanding_client-side_tools/Deployment) article.  
Under the Testing section, in step 7 it’s written:

**In our build script, `npm run test` is called, then you see the string `&&` — this means “if the thing on the left succeeded (exited with zero), then do this thing on the right”. So this translates into: if the tests pass, then build the code.**

but I cannot find it in the package.json file, also not here: [https://github.com/remy/mdn-will-it-miss/blob/master/package.json](https://github.com/remy/mdn-will-it-miss/blob/master/package.json)

should I update the build field under script to be like this?  
“build”: “npm run test && parcel build src/index.html”

Thanks,

---

<div class="post-metadata">

**Author:** ![justsomeone](https://sea1.discourse-cdn.com/flex001/user_avatar/discourse.mozilla.org/justsomeone/32/67787_2.png) [@justsomeone](https://discourse.mozilla.org/u/justsomeone)\
**Post date:** [August 30, 2020, 9:45am UTC](https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668/2 "2020-08-30T09:45:23Z")

</div>

Hello @Rafael_Green

i think so

@chrisdavidmills could you have a look here

have a nice day both of you 🙂

---

<div class="post-metadata">

**Author:** ![chrisdavidmills](https://sea1.discourse-cdn.com/flex001/user_avatar/discourse.mozilla.org/chrisdavidmills/32/21688_2.png) [@chrisdavidmills](https://discourse.mozilla.org/u/chrisdavidmills)\
**Post date:** [October 29, 2020, 12:17pm UTC](https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668/3 "2020-10-29T12:17:47Z")

</div>

Hi @Rafael_Green! Yes, you are right. The `build` command I’ve got in my test version locally is `npm run test && parcel build src/index.html`, but Remy’s version isn’t right.

Do you fancy submitting a PR to him to change this?

---

<div class="post-metadata">

**Author:** ![Rafael\_Green](https://avatars.discourse-cdn.com/v4/letter/r/74df32/32.png) [@Rafael\_Green](https://discourse.mozilla.org/u/Rafael_Green)\
**Post date:** [October 29, 2020, 12:31pm UTC](https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668/4 "2020-10-29T12:31:40Z")

</div>

Hi @chrisdavidmills,  
PR submitted: [https://github.com/remy/mdn-will-it-miss/pull/2](https://github.com/remy/mdn-will-it-miss/pull/2)  
Thanks!

---

<div class="post-metadata">

**Author:** ![chrisdavidmills](https://sea1.discourse-cdn.com/flex001/user_avatar/discourse.mozilla.org/chrisdavidmills/32/21688_2.png) [@chrisdavidmills](https://discourse.mozilla.org/u/chrisdavidmills)\
**Post date:** [October 29, 2020, 12:35pm UTC](https://discourse.mozilla.org/t/understanding-client-side-tools-deployment/66668/5 "2020-10-29T12:35:41Z")

</div>

fabulous, thanks for your help!