Skip to content

Commit ea8cdf0

Browse files
committed
chore(content) Adds more clarifications for the code review assignment
1 parent ecef70b commit ea8cdf0

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

back-end/code-review-exchange/assignment.md

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ The time estimates for the tasks above assume that you are familiar or have prof
5050

5151
Review the code carefully and add any feedback/comments/suggestions you may have.
5252

53-
Please feel free to suggest anything you think that can be improved. Let us know the reasoning behind your suggestions.
53+
Please feel free to suggest anything you think that can be improved. There is no limitation on what you can suggest so add your comments freely. Please do not forget to explain your reasoning behind your suggestions.
5454

5555
### Where can I find the code to be reviewed?
5656

@@ -61,20 +61,23 @@ Please feel free to suggest anything you think that can be improved. Let us know
6161
- Go (Work in progress)
6262
- PHP (Work in progress)
6363
- Ruby (Work in progress)
64-
2. When you accept the assignment a new repo will be created which is the one you will be working on. You have admin rights so you can do anything you want.
65-
2. Click on the Pull Request tab and create a new Pull Request with base branch ```master``` and source branch ```feat/ad-exchange```.
64+
2. When you accept the assignment a new repo will be created which is the one you will be working on. You have admin rights so you can do anything you want. This repo contains a branch called ```feat/ad-exchnage``` which contains the code that you will review.
65+
2. Click on the Pull Request tab and create a new Pull Request with base branch ```master``` and source branch ```feat/ad-exchange```. Use this Pull Request ONLY to add your code review comments.
6666
3. Add your review comments in that Pull Request.
6767

6868
## Task 2
6969

70-
Implement your suggested improvements and implement end-to-end test cases to check that the business requirements are satisfied.
70+
Implement your suggested improvements and end-to-end test cases to check that the business requirements are satisfied.
7171

7272
- You are free to change whatever you want in the entire codebase. You can install any dependencies you might need. There is absolutely no limitation.
7373
- The codebase already contains some (failing) test code and fixtures. Again you can change whatever needs to be changed to make the tests run and verify that the app works as expected. Install anything you need. Change anything you need.
7474
- For your end-to-end test cases you will need to know how to communicate with the bidders. The API documentation for this purpose can be found [here](https://bidderapi.docs.apiary.io).
7575
- Your new changes should be implemented in a separate Pull Request.
76+
- There is no need to implement unit tests, just the end to end tests!
7677

7778
# Deliverables
7879

80+
At the end you should have TWO separate Pull Requests:
81+
7982
1. The original Pull Request which contains your review comments.
8083
2. The new Pull Request that contains your changes. Feel free to add any interesting approaches or important points in the PR's description.

0 commit comments

Comments
 (0)