search

repobee / repobee-sanitizer

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

[misc] Set checkout fetch-depth to 2 #132

Closed slarse closed 3 years ago

slarse commented 3 years ago

This should fix Codecov not always reporting the coverage. It complains about having too shallow fetch depth: https://github.com/repobee/repobee-sanitizer/runs/1962911284?check_suite_focus=true#step:9:25

codecov[bot] commented 3 years ago

Codecov Report

Merging #132 (2592f79) into master (f01dbcc) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #132   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          252       252           
  Branches        43        43           
=========================================
  Hits           252       252

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 f01dbcc...2592f79. Read the comment docs.

slarse commented 3 years ago

@tohanss merge when you've seen what this does

tohanss commented 3 years ago

Before i merge, isnt [test] more appropriate than [misc]? @slarse

slarse commented 3 years ago

I wouldn't say so, this isn't really related to testing. It just fixes a problem with status updates from a particular service provider.