These are chat archives for brunch/brunch

14th
Feb 2017
aturner-andplus
@aturner-andplus
Feb 14 2017 22:24
Where should I direct my with-react skeleton questions?
Aleksey Shvayka
@shvaikalesh
Feb 14 2017 22:25
@aturner-andplus here is OK
aturner-andplus
@aturner-andplus
Feb 14 2017 22:25
Cool, I'm on Linux, when attempting to update my jsx file it's getting compiled to an empty file
Node is v4.7.3, NPM is 2.15.11, I tried removing NVM on the off chance it was relating to it to no luck
I'm also seeing the same thing with the with-react skeleton
Aleksey Shvayka
@shvaikalesh
Feb 14 2017 22:26
we dont support npm < 3
aturner-andplus
@aturner-andplus
Feb 14 2017 22:26
Okay, I'll reinstall nvm and put on the latest node
I tried with node 6.9.4 and NPM 3.10.10
When I had nvm
Node is 7.5.0 and NPM is 4.1.2, doing npm install and npm start now
Yup, same thing
Not sure how to diagnose this :/
Aleksey Shvayka
@shvaikalesh
Feb 14 2017 22:30
please, open an issue on brunch/brunch repo
aturner-andplus
@aturner-andplus
Feb 14 2017 22:38
Done #1672
Anything I can do to help diagnose this issue?
Looking at the stuff I added to the issue it's not entirely clear to me what could be causing this
aturner-andplus
@aturner-andplus
Feb 14 2017 22:48
I think this might be related to vscode not doing atomic saves, I vaguely remember running into this issue before
Is there new documentation on awaitWriteFinish?
It seems to not accept a boolean anymore
we are going to allow pass any options to chokidar soon enough
aturner-andplus
@aturner-andplus
Feb 14 2017 22:51
warn: config.files.awaitWriteFinish: unrecognized key: awaitWriteFinish; expected either of javascripts, templates, stylesheets :(
Is it not supported or something?
Oh
Derp
Hrm
Where do I add awaitWriteFinish to my config file?
Aleksey Shvayka
@shvaikalesh
Feb 14 2017 22:56
brunch/brunch@ab89a01
try master now
aturner-andplus
@aturner-andplus
Feb 14 2017 22:56
I think I got it
// See http://brunch.io for documentation.
exports.files = {
  javascripts: {
    joinTo: {
      'vendor.js': /^(?!app)/,
      'app.js': /^app/
    }
  },
  stylesheets: {joinTo: 'app.css'}
};

exports.watcher = {
  awaitWriteFinish: { stabilityThreshold: 100, pollInterval: 10 }
};

exports.plugins = {
  babel: {presets: ['latest', 'react']}
};
:thumbsup:
Aleksey Shvayka
@shvaikalesh
Feb 14 2017 22:57
yop, it is still in watcher, but it was unintentionally removed from config-validate
so npm brunch warns on it but still respects
@aturner-andplus can we close the issue?
aturner-andplus
@aturner-andplus
Feb 14 2017 23:28
Yup!
Thanks for the help!
Aleksey Shvayka
@shvaikalesh
Feb 14 2017 23:29
You are welcome, please don't hesitate to reach out via gitter or issues