Aidan Feldman
Aidan Feldman
This tool is too big to test manually – it needs [full coverage](https://github.com/education/teachers_pet/blob/master/CONTRIBUTING.md#running-tests). Currently at 95.37%.
Just a little refactor.
In classes that use the [fork](https://education.github.com/guide/forks) strategy, teachers may leave students' pull requests open for review, so that they can submit updates, etc. This may leave them with a bunch...
In teaching [my JS class](https://github.com/advanced-js/syllabus), I've found myself wanting to call all the student's attention to a particular comment, or ensuring everyone sees a particular announcement made through an issue....
This was an initial spike (not ready to merge) – see README diff for the description. Curious to hear if something like this would be useful to anyone else. Thinking...
A first pass... all the `if` statements are feeling ugly, so leaning towards splitting into two Commands with a common subclass: `clone_forks` and `clone_sandboxed_repos` or something – don't love that...
Maybe even default it to `true`? I would think the history of the repo will rarely be needed.
As described in [the guide](https://education.github.com/guide#4-set-up-the-repositories), using forks for assignments is preferable for a lot of reasons. Would be great to have the following actions support forks of assignment repositories: -...
This _should_ be possible, but will take a bit of tinkering with to figure out. Might also be nice to pull out into a dedicated gem/script that can be shared...
https://coveralls.io/r/education/teachers_pet