From 67dc7f1c1ab17534f6bfca1fe857310f5c9e44c5 Mon Sep 17 00:00:00 2001
From: Dmitry Filimonov <dmitry@pyroscope.io>
Date: Thu, 28 Oct 2021 18:38:49 -0700
Subject: [PATCH] build(frontend): disables watch mode, improves error message
 (#482)

* disables watch mode, imporves error message

* fix for snapshot tests
---
 .github/workflows/tests-js-snapshot.yml | 2 +-
 scripts/webpack/webpack.dev.js          | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/tests-js-snapshot.yml b/.github/workflows/tests-js-snapshot.yml
index 781d247fed..70e6f7cf54 100644
--- a/.github/workflows/tests-js-snapshot.yml
+++ b/.github/workflows/tests-js-snapshot.yml
@@ -13,7 +13,7 @@ jobs:
     # if you ever update this container
     # remember to update the container that runs locally
     # (ie ./scripts/jest-snapshots)
-    container: node:current-slim
+    container: node:14.15-slim
     steps:
       - name: Checkout
         uses: actions/checkout@v2
diff --git a/scripts/webpack/webpack.dev.js b/scripts/webpack/webpack.dev.js
index f8a63fd250..e117d1ac16 100644
--- a/scripts/webpack/webpack.dev.js
+++ b/scripts/webpack/webpack.dev.js
@@ -11,7 +11,6 @@ const common = require('./webpack.common');
 const sleep = (ms) => new Promise((res) => setTimeout(res, ms));
 
 module.exports = merge(common, {
-  watch: true,
   devtool: 'eval-source-map',
   mode: 'development',
   entry: {
@@ -27,7 +26,7 @@ module.exports = merge(common, {
         do {
           if (maxTries <= 0) {
             throw new Error(
-              'Could not find pyroscope instance running on http://localhost:4040'
+              'Could not find pyroscope instance running on http://localhost:4040. Make sure you have pyroscope server running on port :4040 (run `make build server`)'
             );
           }
           res = request('GET', 'http://localhost:4040');