9 ms·

This is a serverside pre-PR hook for an analyzer, as I understand it. I was hoping this was a code review tool that allows you to modify the PR without making

This is a serverside pre-PR hook for an analyzer, as I understand it. I was hoping this was a code review tool that allows you to modify the PR without making a commit-merge-push loop, which could have approved changes automagically pulled locally (for the loop). This would save a TON on small edits that many PRs require, including any additional comments that people might want to add to code that come up during PR...modern PRs is where context goes to die.

dolftax 7y ago
We went ahead with integration with providers like GitHub and GitLab to have these checks in a central place as it is the easiest way for a team to adopt a tool like ours. Also, just having a local or IDE plugin doesn't ensure these issues never make it to trunk unless everyone in the team follows it strictly. That said, for the convenience of developers, we're working on the ability to run the analysis and the fixes using our CLI. [1] This opens up doors to use the CLI and build IDE plugins in the near future. [1] https://github.com/deepsourcelabs/cli/issues/15 https://github.com/deepsourcelabs/cli/issues/15