René's URL Explorer Experiment


Title: GitHub - practice-proactive/code-review-tips: :microscope: Common problems to look for in a code review

Open Graph Title: GitHub - practice-proactive/code-review-tips: :microscope: Common problems to look for in a code review

X Title: GitHub - practice-proactive/code-review-tips: :microscope: Common problems to look for in a code review

Description: :microscope: Common problems to look for in a code review - practice-proactive/code-review-tips

Open Graph Description: :microscope: Common problems to look for in a code review - practice-proactive/code-review-tips

X Description: :microscope: Common problems to look for in a code review - practice-proactive/code-review-tips

Opengraph URL: https://github.com/practice-proactive/code-review-tips

X: @github

direct link

Domain: patch-diff.githubusercontent.com

route-pattern/:user_id/:repository
route-controllerfiles
route-actiondisambiguate
fetch-noncev2:55a0d489-980b-a2ee-ecaf-2c3cfbe973f1
current-catalog-service-hashf3abb0cc802f3d7b95fc8762b94bdcb13bf39634c40c357301c4aa1d67a256fb
request-idC76E:1220C7:1903AA:206054:697F6779
html-safe-noncee94dc6577871683b331952cad8bfc4c1dcad3a822a7afd30710d5b2cc9533184
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNzZFOjEyMjBDNzoxOTAzQUE6MjA2MDU0OjY5N0Y2Nzc5IiwidmlzaXRvcl9pZCI6IjgzMjk1ODY0NzA5OTQwMTIwMjUiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ==
visitor-hmac310b9d43fe2062caee9fc088581616d6d0cfb4d0314d977d4d894db4dbe99443
hovercard-subject-tagrepository:118880910
github-keyboard-shortcutsrepository,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location//
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/practice-proactive/code-review-tips
twitter:imagehttps://opengraph.githubassets.com/8d5c33529bb263ea85dc20171fdd82c082aee429d6b9ec47a28c0ba8267eb7fd/practice-proactive/code-review-tips
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/8d5c33529bb263ea85dc20171fdd82c082aee429d6b9ec47a28c0ba8267eb7fd/practice-proactive/code-review-tips
og:image:alt:microscope: Common problems to look for in a code review - practice-proactive/code-review-tips
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
hostnamegithub.com
expected-hostnamegithub.com
None60279d4097367e16897439d16d6bbe4180663db828c666eeed2656988ffe59f6
turbo-cache-controlno-preview
go-importgithub.com/practice-proactive/code-review-tips git https://github.com/practice-proactive/code-review-tips.git
octolytics-dimension-user_id22454976
octolytics-dimension-user_loginpractice-proactive
octolytics-dimension-repository_id118880910
octolytics-dimension-repository_nwopractice-proactive/code-review-tips
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forktrue
octolytics-dimension-repository_parent_id100813255
octolytics-dimension-repository_parent_nworyanmcdermott/code-review-tips
octolytics-dimension-repository_network_root_id100813255
octolytics-dimension-repository_network_root_nworyanmcdermott/code-review-tips
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release7c85641c598ad130c74f7bcc27f58575cac69551
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#start-of-content
https://patch-diff.githubusercontent.com/
Sign in https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpractice-proactive%2Fcode-review-tips
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub SparkBuild and deploy intelligent appshttps://github.com/features/spark
GitHub ModelsManage and compare promptshttps://github.com/features/models
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpractice-proactive%2Fcode-review-tips
Sign up https://patch-diff.githubusercontent.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E&source=header-repo&source_repo=practice-proactive%2Fcode-review-tips
Reloadhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
Reloadhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
Reloadhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
practice-proactive https://patch-diff.githubusercontent.com/practice-proactive
code-review-tipshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
ryanmcdermott/code-review-tipshttps://patch-diff.githubusercontent.com/ryanmcdermott/code-review-tips
Notifications https://patch-diff.githubusercontent.com/login?return_to=%2Fpractice-proactive%2Fcode-review-tips
Fork 0 https://patch-diff.githubusercontent.com/login?return_to=%2Fpractice-proactive%2Fcode-review-tips
Star 0 https://patch-diff.githubusercontent.com/login?return_to=%2Fpractice-proactive%2Fcode-review-tips
MIT license https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/LICENSE
0 stars https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/stargazers
184 forks https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/forks
Branches https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/branches
Tags https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/tags
Activity https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/activity
Star https://patch-diff.githubusercontent.com/login?return_to=%2Fpractice-proactive%2Fcode-review-tips
Notifications https://patch-diff.githubusercontent.com/login?return_to=%2Fpractice-proactive%2Fcode-review-tips
Code https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
Pull requests 0 https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/pulls
Actions https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/actions
Projects 0 https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/projects
Wiki https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/wiki
Security 0 https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/security
Insights https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/pulse
Code https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
Pull requests https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/pulls
Actions https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/actions
Projects https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/projects
Wiki https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/wiki
Security https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/security
Insights https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/pulse
Brancheshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/branches
Tagshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/tags
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/branches
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/tags
30 Commitshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/commits/master/
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/commits/master/
.gitattributeshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/.gitattributes
.gitattributeshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/.gitattributes
LICENSEhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/LICENSE
LICENSEhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/LICENSE
README.mdhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/README.md
README.mdhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/blob/master/README.md
READMEhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
MIT licensehttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#code-review-tips
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#table-of-contents
Introductionhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#introduction
Why Review Code?https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#why-review-code
Basicshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#basics
Readabilityhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#readability
Side Effectshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#side-effects
Limitshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#limits
Securityhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#security
Performancehttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#performance
Testinghttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#testing
Miscellaneoushttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#miscellaneous
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#introduction
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#why-review-code
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#basics
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#code-reviews-should-be-as-automated-as-possible
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#code-reviews-should-avoid-api-discussion
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#code-reviews-should-be-kind
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#readability
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#typos-should-be-corrected
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#variable-and-function-names-should-be-clear
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#functions-should-be-short
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#files-should-be-short
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#exported-functions-should-be-documented
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#complex-code-should-be-commented
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#side-effects
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#functions-should-be-as-pure-as-possible
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#io-functions-should-have-failure-cases-handled
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#limits
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#null-cases-should-be-handled
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#large-cases-should-be-handled
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#singular-cases-should-be-handled
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#user-input-should-be-limited
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#functions-should-handle-unexpected-user-input
And don't rely on client-side validation alonehttps://twitter.com/ryconoclast/status/885523459748487169
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#security
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#xss-should-not-be-possible
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#personally-identifiable-information-pii-should-not-leak
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#performance
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#functions-should-use-efficient-algorithms-and-data-structures
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#important-actions-should-be-logged
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#testing
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#new-code-should-be-tested
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#tests-should-actually-test-all-of-what-the-function-does
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#tests-should-stress-edge-cases-and-limits-of-a-function
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#miscellaneous
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#todo-comments-should-be-tracked
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#commit-messages-should-be-clear-and-accurately-describe-new-code
https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#the-code-should-do-what-its-supposed-to-do
Readme https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#readme-ov-file
MIT license https://patch-diff.githubusercontent.com/practice-proactive/code-review-tips#MIT-1-ov-file
Please reload this pagehttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips
Activityhttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/activity
Custom propertieshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/custom-properties
0 starshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/stargazers
5 watchinghttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/watchers
0 forkshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/forks
Report repository https://patch-diff.githubusercontent.com/contact/report-content?content_url=https%3A%2F%2Fgithub.com%2Fpractice-proactive%2Fcode-review-tips&report=practice-proactive+%28user%29
Releaseshttps://patch-diff.githubusercontent.com/practice-proactive/code-review-tips/releases
Packages 0https://patch-diff.githubusercontent.com/orgs/practice-proactive/packages?repo_name=code-review-tips
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.