Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start of the code review process 👋 #17

Closed
longnd opened this issue Feb 20, 2024 · 3 comments
Closed

Start of the code review process 👋 #17

longnd opened this issue Feb 20, 2024 · 3 comments

Comments

@longnd
Copy link

longnd commented Feb 20, 2024

Hello Tanapon👋 ,
Thank you for your effort on the code submission. I am Long, engineering lead at Nimble, and I am happy to be the reviewer for our code review session.

During the review process, I would like to know more about your decisions, and thus I will create issues where I think there could be more improvements regarding your performance here.

At the same time, please keep in mind that this is a bi-directional process, and I would love to hear back from you as well. Therefore, do not hesitate to raise your questions or share your opinions about the implementation (if any) during the process.

If we are aligned on any issue, and you would like to correct them, please address the issue using a proper git flow (creating a new branch, opening a Pull Request (PR) per issue, and merging the code when you are ready), and I will follow up on those fixes. Please note that you don't have to close any of my created issues after merging your PRs, I will help verify and close them for you once they pass. 😇

We expect the code review process to be completed within 2-3 days at most. As a result, ensure you are responsive during this process. If you need additional time, please inform us right away so that we can plan accordingly.

In the end, I really do hope that you find the process enjoyable. Good luck and happy coding. 🤘

@longnd
Copy link
Author

longnd commented Feb 20, 2024

@tanaponpiti I have completed the initial round of code reviews focusing the most critical concerns to ensure we can keep the whole process within a few days. I am looking forward to hear your feedback on the open issues.

@tanaponpiti
Copy link
Owner

Hello Long,
Thank you for reviewing my code. Hopefully my code is not being too rough, as I have very short amount of time to arrange things due to my current responsibility from day to day . I will try my best to cooperate with the review process as mush as possible.

@longnd
Copy link
Author

longnd commented Feb 21, 2024

@tanaponpiti I have replied to all of the issues. Please feel free to make any changes if you wish. Since we want to move quickly and I have enough understanding of your reasoning behind the chosen solutions, we can conclude this stage of the code review process. Thank you for your efforts thus far. Our TA team will provide feedback via email soon.

@longnd longnd closed this as completed Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants