Peer review functionality to do list for @thereseanders
[x] Use peer_roster_expand() in peer_return()
[x] Vectorize issue creation for multiple paths.
[x] Allow rfeedback and/or afeedback to be missing in issue creation.
[x] Implement difference creation for multiple files.
[x] Create encompassing peer_return()
[x] peer_return() with and without afeedback parameter
[x] Merge upstream changes.
[x] Fix issue message.
[x] Change file to path or local_path, respectively.
[x] Write function to apply labels to all repositories in organization.
[x] Naming: path for file names on GitHub, file for local file paths. This is a bit confusing but the API expects a path parameter.
[x] Separate peer_add_file() and peer_score() into review and rating versions.
[x] Naming: review for aut-to-rev step, rating for rev-to-aut step.
[x] Change messaging of issue - add file name of diff link description. "Read review from ...", "Fill out rating form on the review from ..."
[x] When in the process should labels be created?
[x] Implement issue creation for author to reviewer step.
[x] Naming: separate function names or to and from parameters? For separate function names, should the tag be a2r (author to reviewer) and r2a (reviewer to author). See #59
[x] Are students getting a notification email about issue creation even if they are not watching a repo?
[x] Fix prefix_rev and suffix_rev - if empty, add review to it.
[x] Change functionality to one repo for homework and another repo for review. Every student ends up with 2 repos per assignment. Add review_prefix and review_suffix parameters.
[x] Why is peer_assign() breaking with just 2 reviewers?
[x] Force prefix to be followed by either - or _. Force suffix to be lead by either - or _.
[x] Fix suppressMessaging() for issue creation (glue::glue("{y}/{path}")) - maybe use paste0 instead?
[x] Double check peer_read_roster() and peer_expand_roster(). Is the fname argument needed anymore? Can it be created within the scoring functions and therefore deleted from peer_read_roster(). If yes, the output and all dependent functions need to be changed.
[x] Update figure for paper with new function names.
[x] Merge upstream master changes and difference in function calls
[ ] Naming: "Move" for GitHub-to-Github actions, "add" for local machine to GitHub actions
[x] Update vignette
[x] Complete documentation
[x] Finish instructions to students
[x] Can labels be created per organization (rather than per repo)? As tidyverse team! Look into tidylabels.
[ ] Modify path and/or file to afeedback or rfeedback for file names of these forms within peer_create_rform(), peer_create_aform(), new versions of peer_score
[x] Message suppression for peer_assign_clone()
[ ] Sample task from Mine
[x] Set up test run example for RStudio Interns
[x] Compute the exact amount of API calls
[x] screen shots for vignette?
Later point
[ ] peer_return_clone() merge conflict - force overwrite?
[ ] Implement stripping of grades as a parameter. strip_score = FALSE as default.
Peer review functionality to do list for @thereseanders
peer_roster_expand()inpeer_return()rfeedbackand/orafeedbackto be missing in issue creation.peer_return()peer_return()with and withoutafeedbackparameterpathfor file names on GitHub,filefor local file paths. This is a bit confusing but the API expects apathparameter.peer_add_file()andpeer_score()intoreviewandratingversions.toandfromparameters? For separate function names, should the tag bea2r(author to reviewer) andr2a(reviewer to author). See #59prefix_revandsuffix_rev- if empty, add review to it.review_prefixandreview_suffixparameters.peer_assign()breaking with just 2 reviewers?-or_. Force suffix to be lead by either-or_.suppressMessaging()for issue creation (glue::glue("{y}/{path}")) - maybe usepaste0instead?peer_read_roster()andpeer_expand_roster(). Is thefnameargument needed anymore? Can it be created within the scoring functions and therefore deleted frompeer_read_roster(). If yes, the output and all dependent functions need to be changed.pathand/orfiletoafeedbackorrfeedbackfor file names of these forms withinpeer_create_rform(),peer_create_aform(), new versions ofpeer_scorepeer_assign_clone()Later point
peer_return_clone()merge conflict - force overwrite?strip_score = FALSEas default.