search

open-rmf / rmf_traffic

Traffic management libraries for RMF
Apache License 2.0
28 stars 26 forks source link

Fix upload-artifact@v2 paths #48

Closed aaronchongth closed 3 years ago

aaronchongth commented 3 years ago

Signed-off-by: Aaron Chong aaronchongth@gmail.com

Bug fix

Fixed bug

The github action upload-artifact@v2 finds the most common base directory and attempts to upload them. latest_test is a symlink so it fails when it needs to upload the latest_test directory. This just changes it to upload the whole log directory.

Reference: https://github.com/actions/upload-artifact

codecov[bot] commented 3 years ago

Codecov Report

Merging #48 (20a3c4c) into main (6eb265e) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #48   +/-   ##
=======================================
  Coverage   22.12%   22.12%           
=======================================
  Files         354      354           
  Lines       29213    29213           
  Branches    13478    13478           
=======================================
  Hits         6464     6464           
  Misses      13548    13548           
  Partials     9201     9201
Flag Coverage Δ
tests 22.12% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.