difflens · HackerTrans

3 ปีที่แล้วdiscuss

I built DiffLens (https://www.difflens.com/) initially just for myself. It's a diff tool that uses abstract syntax trees to make the diff more review-able. It's free for anyone to use too. I use it every day to review my diffs. If anyone works on Typescript, Javascript, HTML and/or CSS, do check it out!

difflens

3 ปีที่แล้วdiscuss

I'm the author of DiffLens (https://www.difflens.com/). I initially built it for myself too (and use it everyday) and it's currently free for anyone to try. It's an attempt to use abstract syntax trees to make diffs more readable. Happy to see another diff project here!

difflens

4 ปีที่แล้วdiscuss

Across the different teams and orgs I have worked at, there's always been 1 constant during code reviews: Each comment provided an explanation of why some change was being debated. Every engineer knew the review is not a judgement of their coding prowess. Rather each discussion was purely on the merits of that line of code. Sure, there can be disagreements, but it was never judgemental. Usually folks are amenable when there is good justification to the feedback in my experience. It goes both ways too: I personally don't care if some of my nits are not fixed.

Btw, since I think it is relevant to this discussion, we're working on making code reviews deeper than text based diffs. Check out (DiffLens)[https://github.com/marketplace/difflens] if you're code base is primarily TS, JS and/or CSS to get language aware diffs on your GitHub pull requests. We've found that it makes understanding code changes much easier

comments