-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Remove @testable
imports WIP
#8145
base: main
Are you sure you want to change the base?
Conversation
@testable
imports WIP
@swift-ci test |
@swift-ci test |
@swift-ci test |
@swift-ci test windows |
Sorry for the negative feedback, but this seems like a really bad idea. it opens all of these APIs to be used by anything in package for the (imo small) benefit of avoiding |
This PR is a draft WIP and isn't ready for feedback. I'm only trying it out to see if this enables incremental builds for bootstrap builds at all via any changes in code, which may not be the case in the end. At that point we won't care if |
@swift-ci test linux |
678769d
to
f919e2c
Compare
@swift-ci test |
@swift-ci test |
@swift-ci test windows |
[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]