Skip to content

Commit

Permalink
Replaced npm start command with direct webpack invocation
Browse files Browse the repository at this point in the history
Because npm is not a process manager, it does not propagate SIGINTs to the scripts it starts
(webpack in this case) - thus, doing Ctrl+C on the Foreman process will kill the npm process
but not the webpack process.  Replacing the "npm run start" command with whatever is
listed under "start" in package.json bypasses this signal handling discrepancy.
  • Loading branch information
alisawallace committed Sep 11, 2023
1 parent 3a0bfe4 commit 7d5e357
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Procfile
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
web: bundle exec puma
worker: bundle exec sidekiq -r ./worker.rb
webpack: npm run start
webpack: ./node_modules/.bin/webpack serve --config ./webpack.config.js --mode=development

0 comments on commit 7d5e357

Please sign in to comment.