Ask HN: Are We Approaching Code Reviews Wrong?

2 points

2 years ago

As a tech lead, I recently tracked my team's time and found we spent an average of 12.5 hours per developer per week on PR reviews(include discussion of PR review results) - more than half of our coding time! This led me to question our approach to code reviews.

Key challenges we usually face:

- Are we wasting senior developers' time? How do we balance thorough reviews with efficient use of expertise?

- Ensuring consistent review quality across large teams and time zones

- Conducting comprehensive reviews in small teams without slowing development

- Focusing on the right aspects: architecture, logic, or style

- Using PR reviews for knowledge sharing and mentorship effectively

--------------------------------

I've been experimenting with AI-assisted PR reviews using Graph RAG for context understanding. The idea was to build a knowledge graph of our codebase that could make LLM understand Code relationships and dependencies. Early results show promise in reducing initial review time and catching consistent patterns, but it raises new questions about the balance between AI assistance and human expertise.

I'd love to hear your thoughts:

  1. What's your team's current review process? What works well, and what doesn't?

  2. How do you measure the effectiveness of your review process?

  3. What kind of support do you think an ideal AI PR review tool should provide? Comprehensive analysis, focused insights, or specialized in certain aspects?

If you're curious about AI-Powered PR review tool, I've published one called LlamaPReview that we've been using. It's still in beta, but feel free to check it out: https://github.com/marketplace/llamapreview/

Let's discuss how we can make code reviews more effective, efficient, and valuable for our teams!