This content originally appeared on DEV Community and was authored by Minsu Kim
Introduction
In the previous weeks, I had change to contribute internal Seneca opensource projects: fix the bug on Telescope and IPC144 course note renewal. The last piece of task is code reviewing on other student's PR. The code reviews that I have done are PR1 and PR2
PR1 Code Review
The first code review is "ASCII Collating Sequence image converted into Table" which is very similar with my IPC144 course note renewal. The reason why I had code review on this PR is that I want to know and compare how he resolved the issue.
This is Markdown code what he has done. He has smart idea to add one more column to improve readability on Docusarus. However, not only improving readability but also it helps developers to edit code for maintenance. He used prettier to align all the content in right position. I had fixed my code after I have reviewed his code.
PR2 Code Review
The second code review is "Audit and Fix arrays.md". The comment that I left for the code review is that there is a picture for explaining the concept of array. However, the picture did not have proper alt tag in <img> tag.
The alt tag provides context of image that display and inform for search engine crawler which is important for SEO. Also alt tag is important for accessibility perceptive
Conclusion
The code review is not only helpful for the author of the code but also it is helpful to remind me the right knowledge and provide me better idea to solve the issue.
This content originally appeared on DEV Community and was authored by Minsu Kim
Minsu Kim | Sciencx (2021-11-18T13:29:45+00:00) Code Review. Retrieved from https://www.scien.cx/2021/11/18/code-review/
Please log in to upload a file.
There are no updates yet.
Click the Upload button above to add an update.