Urgent.News

600+ sources. One page. See who else covered it.

Editions

Tech

Stop sending me huge PRs; a rant

Tired of dealing with massive PRs, a developer expresses frustration about the excessive size of code changes. They argue that small, manageable PRs are easier to write and review, benefiting both the reviewer and the maintainers. The developer believes that the true purpose of small PRs is to provide digestible, reviewable, and understandable sections of work, rather than creating a single, enormous piece of code to be digested all at once.

They suggest that the time needed to fully comprehend a piece of code increases exponentially with the number of lines it contains, causing reviewers to waste an increasing amount of time on larger PRs. The developer questions the need for lengthy comments, stating that proper naming conventions and existing documentation tools (such as jsdoc, rustdoc, and javadoc) are sufficient for understanding code.

For those who advocate using AI to assist in code comprehension, the developer counters that AI tools are already capable, and using them to generate code that requires further human review defeats the purpose. They question the motivation behind submitting large PRs to reviewers, suggesting that it may be a tactic to force approval. In conclusion, the developer urges others to adopt smaller, more manageable PRs to save time and improve code review efficiency.

Written by urgent.news from Hacker News's reporting — not their text. Machine-written — may contain errors; check the original before relying on it.

Read the original at getsmall.xyz →

More in Tech

Talks to sell PayPal to Stripe and Advent are heating up

PayPal is still reportedly negotiating a potential sale to Stripe and private equity firm Advent, as the fintech firm's new CEO attempts to turn the company around.

  • Talks to sell PayPal intensify between Stripe and Advent
  • PayPal rejected initial $60.50 per share offer in July
  • Negotiations ongoing, potential deal near future

More from Friday 14 August →