-
-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use ReTest #84
Use ReTest #84
Conversation
Codecov Report
@@ Coverage Diff @@
## master #84 +/- ##
=======================================
Coverage 90.30% 90.30%
=======================================
Files 11 11
Lines 629 629
=======================================
Hits 568 568
Misses 61 61
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@jakobnissen So this is a bummer How married to the nesting testsets are you? I could preserve it by making each of the sub-files a new module (eg TestFASTA.IO - it would be a bit of a hassle since each thing would need to re-do imports etc. Another option is to just include the name in all of the testsets (eg I'm partial to the last one. You can pass multiple filters, so eg if you want the "basic" tests, but only the ones from "IO", you can do |
Argh, actually, that doesn't work as well because it doesn't pick up on changes to the |
I don't really have any opinions, since I haven't used ReTest for any real project yet. |
801ffb4
to
d4bbb9f
Compare
Alright, I think this should work. I've added a |
Hm... don't see why julia 1.6 would be different here. |
Oh, the problem is windows/mac, not the julia version 🤔 I only have linux box, anyone with mac/windows have a thought? |
It could be having different files whose names differ by case only? I think Windows paths are case insensitive |
@jakobnissen OK, I think this is good to go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Real nice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Real nice.
Branched off of #68 . Closes #83