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

Do not store results affected by crashes in the cache #2292

Merged
merged 1 commit into from
Oct 4, 2015

Conversation

jonas054
Copy link
Collaborator

@jonas054 jonas054 commented Oct 3, 2015

When there's an error in a cop for a certain inspected file, we don't want to cache the result, because that means the error will be hidden in the next run.

When there's an error in a cop for a certain inspected
file, we don't want to cache the result, because that
means the error will be hidden in the next run.
@jonas054 jonas054 force-pushed the do_not_cache_after_crashing branch from b5edf83 to f0404c7 Compare October 3, 2015 17:23
bbatsov added a commit that referenced this pull request Oct 4, 2015
Do not store results affected by crashes in the cache
@bbatsov bbatsov merged commit e24b96a into rubocop:master Oct 4, 2015
@jonas054 jonas054 deleted the do_not_cache_after_crashing branch October 4, 2015 06:23
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

Successfully merging this pull request may close these issues.

2 participants