nimble icon indicating copy to clipboard operation
nimble copied to clipboard

Use walkDirRec for nimble test, output to build/tests

Open LiamDobbelaere opened this issue 4 years ago • 4 comments

This is not meant to be added into master directly, there's probably a lot wrong with this code style-wise but it works for me. Sorry about the whitespace, I do use nimpretty so maybe the original file wasn't formatted?

It serves as an illustration to this issue: #849

This is how I expected nimble test to actually work, not whatever it is doing now.

LiamDobbelaere avatar Sep 12 '20 14:09 LiamDobbelaere

CI is failing since this functionality is explicitly disabled - see tests/testCommand/testsIgnore.

Best to wait for @dom96 to confirm why this is not supported. It might very well be a breaking change but seems like a reasonable ask as well.

genotrance avatar Sep 12 '20 17:09 genotrance

Thanks, don't mean to be a bother. I was just starting a project in Nim and this particular thing just bothered me is all.

LiamDobbelaere avatar Sep 12 '20 22:09 LiamDobbelaere

What's the status of this? Can we get those formatting changes removed?

dom96 avatar Feb 03 '21 22:02 dom96

ping

dom96 avatar Jul 04 '21 15:07 dom96