code reviews
Developer feedback sessions
Code reviews are examinations of software code by developers other than the original author, typically before the code is merged into a shared codebase.
The practice catches bugs, improves code quality, and spreads knowledge across a team: Can you do a code review on my pull request?
Reviews range from quick approvals to extended back-and-forth discussions about design decisions.
Ease of understanding text
- maintainabilityEase of ongoing repair
- correctnessState of being accurate
- securityProtection from threat
- performanceCarrying out a task
scalability · standards · modularity · consistency · documentation · testability · complexity · duplication
Spoken or written reactions
- suggestionsProposed ideas or advice
- critiqueDetailed evaluative analysis
- nitpickFind petty faults
- actionableAble to be acted upon
constructive · inline · threaded · approval · blocking · non-blocking · tone · specificity
Joint effort toward shared aim
- knowledge-sharingDistributing learned information collectively
- discussionExchange of ideas
- mentoringAdvising someone less experienced
- onboardingNew member integration process
communication · consensus · disagreement · escalation · remote · synchronous · asynchronous
Advising someone less experienced
- patternsRecurring arrangements or designs
- anti-patternsCounterproductive common solutions
- growthIncrease in size
- expertiseSkill from extensive practice
readability
Ease of understanding text
- maintainabilityEase of ongoing repair
- correctnessState of being accurate
- securityProtection from threat
performance · scalability · standards · modularity · consistency · documentation · testability · complexity · duplication
comments
Spoken or written reactions
- suggestionsProposed ideas or advice
- critiqueDetailed evaluative analysis
- nitpickFind petty faults
actionable · constructive · inline · threaded · approval · blocking · non-blocking · tone · specificity · rationale
teamwork
Joint effort toward shared aim
- knowledge-sharingDistributing learned information collectively
- discussionExchange of ideas
- mentoringAdvising someone less experienced
onboarding · communication · consensus · disagreement · escalation · remote · synchronous · asynchronous
mentoring
Advising someone less experienced
- patternsRecurring arrangements or designs
- anti-patternsCounterproductive common solutions
- growthIncrease in size
approval
Favorable opinion given
branch · reviewer · author · iteration · turnaround · blocking · sign-off · gate · workflow · automated
- tech culturedevelopers, coders, startup culture, programmers, computers
- Tech Presentationskeynote, keynotes, TED talk, demo, demos
- open sourceGNU, open source software, free software, GPL, forks
- publicationsbooks, book, newspaper, magazines, magazine
- Agile Developmentagile, XP, refactoring, iteration, sprint
code reviewsworking in isolation
review, code, debugging, peer review, source code, software development, code review