Skip to content
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

Guard::Brakeman failed to achieve its <run_all> #21

Open
jaredmoody opened this issue Oct 15, 2013 · 1 comment
Open

Guard::Brakeman failed to achieve its <run_all> #21

jaredmoody opened this issue Oct 15, 2013 · 1 comment

Comments

@jaredmoody
Copy link

Started getting this error since upgrading to guard 1.8.3, gist with backtrace here:

https://gist.github.com/jaredmoody/6984854

I'm not sure what changed in guard that's causing @tracker to be null during run_all, but adding @tracker ||= ::Brakeman::Scanner.new(@scanner_opts).process above line 58 seems to have fixed it for me, but probably isn't the right thing to do :P

@oreoshake
Copy link

I think the only downside to that is that it would have to reprocess the entire app each time run_all is run. Not ideal for an app that takes 6 minutes to process. I have no idea why that instance var would disappear. I'll see what's different in guard. Did you do a 1.8.2 -> 1.8.3 upgrade or are there more version to check?

Sidenote: I have no idea why return run_all unless @tracker.checks is in run_on_changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants