-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Wallaby errors on stop and restart. #66
Comments
Reproduced, will be fixing it shortly. A better workaround for now is to set |
Fixed. |
@ArtemGovorov Is this fixed? If I don't disable instrumenting for HTML files, I still get a similar error in WebStorm:
|
@behrangsa Could you please share your wallaby.js config file? |
Hi Artem, If you give me your email I can send it to you tomorrow. Cheers, On Mon, 15 Jun 2015 at 5:58 pm Artem Govorov notifications@github.com
|
@behrangsa please send it to hello@wallabyjs.com |
What is the status of this? |
The original issue was fixed back in April. If the issue is reproducible, please share a sample repository with a wallaby configuration where I could reproduce it to fix it. |
Hi @ArtemGovorov, Apologies for the very long delay. Just tried v1.0.81 and this is not happening anymore. So no need to reopen it. Thanks, |
It is now working well for me as well. Thanks! |
I have the same issue. It happens everytime when I restart Visual Studio and try to start Wallaby until I make any change in |
@aanklewicz92 it is an error in one of the third party node modules that wallaby.js is using (happens mostly on Windows), I'm not sure why it happens and why changing wallaby config affects it. Try stopping wallaby and then killing all phantomjs and iojs processes if there're any. |
@aanklewicz92 while I'm investigating the issue reason, please let me know if you spot any phantomjs/iojs processes after you stop wallaby.js (unless you have other apps spawning those processes). |
I have made some changes in core v.1.0.119 that should hopefully make the issue go. Please let me know if you still see and I'll keep chasing down. |
For now I haven't got this problem |
I just upgraded Wallaby and it is still showing this error when it starts up:
It doesn't prevent tests from running, but it is a bit disappointing to say the least. IDE: WebStorm 10.0.4 |
Actually it is not working fine either. Changes are not picked up, tests are not automatically executed, files are not annotated, etc. |
@behrangsa Did it stop working after the update and was working before? Please restart your IDE, update to suggested version again and start wallaby. Let me know if it starts working or not. |
I hadn't used it for a while so I don't know if it was working before or not. Restarting the IDE didn't make any difference either. |
Please send me idea.log (IDE - menu Help - Show Log...), also in your config file please add debug flag, restart wallaby and send me a few lines at the end of the Wallaby Console Output: module.exports = function () {
return {
debug: true,
workers: { ... |
Also try stopping wallaby and killing all phantomjs/iojs processes after you stop wallaby.js if there're any (unless you have other apps spawning those processes), then start it again. |
@behrangsa Re: version - it's expected, latest plugin version is 1.0.36 and latest core version is 1.0.127. |
I just deleted |
Deleting didn't fix the problem, but the session error is now only reported once:
|
Thanks Behrang, the error message is harmless, it's just a third party module exception that doesn't affect anything, so you may ignore it for now. I'll keep investigating how to remove it. Let's move the conversation to email and try to find out why 1.0.36 doesn't show coverage and doesn't do change tracking. |
This is a strange one... running Wallaby.js works perfectly the fist time with no errors. Stopping Wallaby and re-running gives the following error:
Running again gives a different error:
I've had a few different combinations of the error...
My workaround is when I edit the
wallaby.conf.js
file and run Wallaby again, everything works fine. For example, if I add the following bogus line into myfiles
array:{ pattern: 'something-totally-bogus', ignore: true }
. I just have to continuously change this bogus value when stopping and starting wallaby.The text was updated successfully, but these errors were encountered: