noobling / Homefornow

Current beta website
https://www.homefornow.org.au
MIT License
8 stars 3 forks source link

Admin Dashboard #47

Closed jamliii closed 6 years ago

noobling commented 6 years ago

@jamliii Please in the future wait for someone else to merge the pull request that way someone can do a code review and give you feed back on where to improve and possibly to pick up bugs. That is the whole point of pull requests ahah. Read more here: https://www.atlassian.com/agile/software-development/code-reviews This point will be most relevant to you

Code reviews mentor newer engineers A special aspect of agile is that when new members join the team more seasoned engineers mentor the newer members. And code review helps facilitate conversations about the code base. Often, teams have hidden knowledge within the code that surfaces during code review. Newer members, with fresh eyes, discover gnarly, time-plauged areas of the code base that need a new perspective. So, code review also helps ensure new insight is tempered with existing knowledge.