Skip to content

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Aug 14, 2025

Continuing the effort started in #2228.

@ahoppen
Copy link
Member Author

ahoppen commented Aug 14, 2025

@swift-ci Please test

package import SourceKitLSP
import SwiftLanguageService

#if canImport(DocumentationLanguageService)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we not need to guard the DocumentationLanguageService use down below?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do. Apparently I also added the check in a future commit that just didn’t make it into this PR 😂

@ahoppen ahoppen force-pushed the move-docc-service branch from 5f209ae to fc80a66 Compare August 14, 2025 18:47
@ahoppen
Copy link
Member Author

ahoppen commented Aug 14, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Aug 14, 2025

@swift-ci Please test Windows

@ahoppen ahoppen force-pushed the move-docc-service branch from fc80a66 to de02f60 Compare August 15, 2025 09:49
@ahoppen
Copy link
Member Author

ahoppen commented Aug 15, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Aug 15, 2025

@swift-ci Please test Windows

@ahoppen ahoppen force-pushed the move-docc-service branch from de02f60 to aeffb2e Compare August 15, 2025 17:14
@ahoppen
Copy link
Member Author

ahoppen commented Aug 15, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Aug 15, 2025

@swift-ci Please test Windows

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented Aug 16, 2025

@swift-ci Please test Windows

@ahoppen ahoppen force-pushed the move-docc-service branch from aeffb2e to 615dead Compare August 17, 2025 20:22
@ahoppen
Copy link
Member Author

ahoppen commented Aug 17, 2025

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Aug 17, 2025

@swift-ci Please test Windows

@ahoppen ahoppen merged commit 10c8991 into swiftlang:main Aug 18, 2025
3 checks passed
@ahoppen ahoppen deleted the move-docc-service branch August 19, 2025 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants