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
Fix up Imports #2963
Fix up Imports #2963
Conversation
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.
It's kinda sad just how inaccurate the API breakage checker can be about some things...
with: | ||
fetch-depth: 0 | ||
- name: Build | ||
run: swift build -Xswiftc -DBUILDING_DOCC |
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.
-D
flags can't be passed directly to swift build
? I could've sworn that was supported.
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.
Honestly I didn't actually try! I'll try in the next repo and we can always backport it
@@ -37,3 +37,13 @@ jobs: | |||
with_coverage: false | |||
with_tsan: false |
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.
While we're at it, we should either turn off TSan for the provider tests or turn it on for the unit tests.
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.
It should probably be on really. I'll make a note to enable it when I do some Sendable work
@gwynne We should probably raise issues for all these API checker problems... 😅 |
Fixes up the imports in Vapor so it will build without exporting everything.
This allows us to avoid exports when building documentation and future proofs us.
Also run this through CI to avoid PRs breaking the docs