search

MobileNativeFoundation / swift-index-store

Library to read from Swift / clang source code indexes
Apache License 2.0
137 stars 12 forks source link

Remove module name replacement #38

Closed keith closed 1 year ago

keith commented 1 year ago

I don't understand why we added this, the context was:

the previous where testableImports.contains(dependentUnit.moduleName) would skip over the test files because test files would never show up in testableImports i.e. we don't ever see @testable import FooTests.

But I don't get that reasoning because you shouldn't ever be testable importing your own module or any other Tests module, and if you have a naming scheme where you do I don't think you want to skip them.