This workshop is part of eas-coding-workshops workshop series.
- Brock to create and push skeleton functions for everyone
-
(optional, but highly recommended) Ensure you have a github ssh key authentication connection set up.
-
Clone this repo and create an RStudio or project from it.
-
RStudio
- In top right corner click ont he name of your current project (or "(no project)").
- Chose "New Project">"Version Control">git
- Paste
[email protected]:livingingroups/telephone2024fall.git
-
If using R, "Load all"
-
If using R, make sure you can run your the
telephone(word)
function and your function (your_name(word)`. -
Make sure you can find your personal R or file.
- R: in the
R
directory
- R: in the
-
Determine who will review who
-
Conceptual explanation of telephone game
-
Conceptual explanation and diagram of what is happening with git, github and locally in the next 3 sections.
-
Create a new branch
-
Make changes to your function (repeat as necessary):
-
Make a change
-
Test your function
-
Run Load All
-
In the console, run your function with whatever test inputs you chose.
-
-
-
Once you're happy, Commit your change
- RStudio: Commit button in Git tab
-
Push your change
- Rstudio: Push Button in Git tab
-
Create a pull request
-
Switch over from RStudio to Github, open https://github.com/livingingroups/telephone2024fall
-
You will likely see a prompt to create a PR from your recently pushed branch.
-
If you don't, select your branch from the dropdown.
-
-
In Rstudio or with git command line, "Pull" from remote.
-
Switch to the branch of the person you're reviewing.
-
Run Load All
-
Test out their function (
their_name(word)
) -
If it works, find their pull request here: https://github.com/livingingroups/telephone2024/pulls and click Approve. You can then click "merge".
-
If it doesn't work, let them know...they can update and try again.
⬜ Whiteboard explanation
-
Pull
-
Checkout main
-
Create a new branch
yourname-theirname-comments
-
Open their R file, look at what they've done and make 2-3 suggestions.
-
Commit
-
Push
-
Make a PR (set it to Draft status). - Your comments will show up as the diff.
-
Ideally, testing would be part of this. If the author had written tests, it would speed up the reviewer's process.
-
Whiteboard: Explanation of forks/multiple remotes.