CragVS Code extension
Reviews your pull requests before anyone else has to.
Review a Whole BranchAnonymousI want one review across every commit on my branch before I open the pull request, not a separate one per file as I go.Support Team Sensible, and it is a different job from the per-file review rather than a bigger one: things worth saying about a branch are usually about how the changes fit together. Thinking it through before we commit to it.Explain the FindingMeeraSome findings I understand at a glance. Some I read three times and still cannot tell whether it matters. A sentence on why it was flagged would settle most of them.Support Team In progress. Each finding gets a short why, and where the reasoning depends on something elsewhere in the file it will link to that line rather than describing it.Our Rules, Not YoursRaviWe have a written style guide our team actually follows. Crag doesn't know about it, so it flags things we've decided are fine and misses the things we care about. A file in the repo it reads would do it.Support Team Planned, and it is the right shape: a file in the repo, reviewed like any other file, rather than settings in an account somebody left the company with. Rules will be able to loosen as well as tighten, because half of what a team writes down is which warnings not to bother with.Auto-Push FixesAnonymousif crag can see the problem it may as well fix it and commitSupport Team We're going to say no, and it's worth saying why rather than leaving it open. Crag reviews. The moment it also writes to your branch, the thing reviewing the code and the thing changing it are the same thing, and the review stops being independent. That's the whole value of it. It'll keep offering the fix as a diff you apply in one click. Same number of keystrokes, no conflict of interest.Only the DiffLenaOn an old file it reviews the whole thing and I get forty findings about code I didn't touch. I want to know about the twelve lines I changed.Support Team Being told about a decade of somebody else's decisions when you fixed a typo is a good way to get an extension uninstalled. Agreed.