Skip to content

Commit 70f4890

Browse files
authored
Merge pull request #2 from vincentsels/clarify-pr
Clarify the PR part
2 parents 75a1fb9 + 8ce999d commit 70f4890

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

Week1/MAKEME.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ Because we're actually going to build an entire application, this homework is mo
1010

1111
Over the coming weeks, we are going to write an application that uses the [GitHub API](https://developer.github.com/guides/getting-started/).
1212

13-
This application should display information about the available [HYF GitHub repositories](https://github.com/hackyourfuture). Eventually, this will be the functionalities:
13+
This application should display information about the available [HYF GitHub repositories](https://github.com/hackyourfuture). Eventually, it should have this functionality:
1414

1515
- The user should be able to select a repository from a list of available repositories.
1616
- The application should display high-level information about the selected repository and show a list of its contributors.
@@ -158,4 +158,8 @@ If the answer is 'yes' to the preceding questions you are ready to follow these
158158
git push -u origin week1
159159
```
160160

161-
- [ ] Create a pull request for your `week1` branch against our class' (Rob's) fork.
161+
- [ ] Create a pull request (PR) for your `week1` branch against our class' (Rob's) fork `master` branch.
162+
163+
- [ ] Keep an eye on your notifications: if there are issues with your homework, we'll add comments to your PR. If we don't add comments, that means you've done an excellent job - or we dind't find the time to review it :(.
164+
165+
- [ ] You can always amend your homework by pushing more commits to your branch - they'll be included in the pull request. Do this to resolve any remarks we gave in the comments!

0 commit comments

Comments
 (0)