search

repobee / repobee-sanitizer

A plugin for sanitizing master repositories before distribution to students
MIT License
2 stars 3 forks source link

Test plug.Error raised when filelist does not exist #36

Closed tohanss closed 4 years ago

tohanss commented 4 years ago

Fix #33

codecov[bot] commented 4 years ago

Codecov Report

Merging #36 into master will increase coverage by 1.47%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #36      +/-   ##
===========================================
+ Coverage   98.52%   100.00%   +1.47%     
===========================================
  Files           6         6              
  Lines         136       136              
  Branches       23        23              
===========================================
+ Hits          134       136       +2     
+ Misses          1         0       -1     
+ Partials        1         0       -1
Impacted Files Coverage Δ
repobee_sanitizer/_sanitize_repo.py 100.00% <0.00%> (+3.12%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a6bae31...299bc90. Read the comment docs.

slarse commented 4 years ago

Okay so now we just need to fix the confusion here, because this PR contains commits that it should not. The only commits that we actually want here are 6206e4b and 299bc90

So, what you want to do is to close this PR, then check out a new branch from a synced master, and cherry pick those commits onto the new branch. Then open a new PR.