colcon / colcon-clean

A colcon extension to clean package workspaces
http://colcon.readthedocs.io
Apache License 2.0
46 stars 4 forks source link

Exclude test directory during install #13

Closed timonegk closed 2 years ago

timonegk commented 2 years ago

Currently, because packages = find: is used and the test folder also contains a __init__.py, the test folder is also installed. This is unintended and leads to problems with some other packages. Adding the exclude option solves the problem.

codecov-commenter commented 2 years ago

Codecov Report

Merging #13 (ad27949) into master (a7f1074) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #13   +/-   ##
======================================
  Coverage    7.72%   7.72%           
======================================
  Files          11      11           
  Lines         272     272           
  Branches       41      41           
======================================
  Hits           21      21           
  Misses        251     251           

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 a7f1074...ad27949. Read the comment docs.

ruffsl commented 2 years ago

Nice catch @timonegk , thanks!