Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Run only on changed code #69

Closed
eyaleizenberg opened this issue May 24, 2017 · 2 comments · Fixed by #83
Closed

Run only on changed code #69

eyaleizenberg opened this issue May 24, 2017 · 2 comments · Fixed by #83

Comments

@eyaleizenberg
Copy link

Hi,
Is there a way to have the code climate linter run only on changed code (similar to the way it runs on the web app) rather than analyzing the whole file?

@maxjacobson
Copy link
Contributor

Hi @eyaleizenberg. This isn't currently possible. It would need to be supported by the CLI before it would be supported by the Atom package. It's something we've thought about in the past, but haven't yet prioritized.

@cgalvarez
Copy link
Contributor

Hi @eyaleizenberg. I think that approach would make some engines to provide false results. Think of the duplication engine, which could not find duplicated code inside a file if you just send the modified chunk of code, or calculating the complexity of a function with phpmd if you change some of its lines and only send them.

While some engines could not be affected, there are others that could, and as @maxjacobson said, this should be a feature implemented by each engine and ultimately by the CLI.

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

Successfully merging a pull request may close this issue.

3 participants