search

repobee / repobee-sanitizer

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

[feat] Add discover mode for sanitize_repo #54

Closed tohanss closed 4 years ago

tohanss commented 4 years ago

sanitize-repo now has the --discover-files argument. As opposed to --file-list, this lets us find any file that has ANY marker in it and send this forward to be sanitized (and syntax checked first of course).

Fix #32

codecov[bot] commented 4 years ago

Codecov Report

Merging #54 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #54   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          146       162   +16     
  Branches        24        28    +4     
=========================================
+ Hits           146       162   +16
Impacted Files Coverage Δ
repobee_sanitizer/_sanitize.py 100.00% <100.00%> (ø)
repobee_sanitizer/_sanitize_repo.py 100.00% <100.00%> (ø)

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 d80931c...0239022. Read the comment docs.

tohanss commented 4 years ago

Now we can go trough how to properly sove merge conflicts!

slarse commented 4 years ago

@tohanss Ok to merge or do you want to fix those type hints?