Skip to content

Remove explicit swift-docc-plugin dependency#354

Merged
iCharlesHu merged 1 commit into
mainfrom
ktoso-patch-1
Jul 14, 2026
Merged

Remove explicit swift-docc-plugin dependency#354
iCharlesHu merged 1 commit into
mainfrom
ktoso-patch-1

Conversation

@ktoso

@ktoso ktoso commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Nowadays (as weird as it is) projects are not expected to have the dependency declared explicitly. It must be added ad-hoc or by the swift package index. Otherwise all kinds of tools break, including swiftlang/docs.

It would be ideal that all projects can just add the plugin, but that's not the state of the world nowadays.

Meanwhile I did workarounds in adopter projects to unbreak swiftlang/docs building with those added, more details here swiftlang/swift-java#845

cc @heckj

Here's the related swiftpm issue swiftlang/swift-package-manager#10294

Nowadays (as weird as it is) projects are not expected to have the dependency declared explicitly. It must be added ad-hoc or by the swift package index. Otherwise all kinds of tools break, including swiftlang/docs.

It would be ideal that all projects can just add the plugin, but that's not the state of the world nowadays.

Meanwhile I did workarounds in adopter projects to unbreak swiftlang/docs building with those added, more details here swiftlang/swift-java#845

cc @heckj
@ktoso ktoso requested a review from iCharlesHu as a code owner July 14, 2026 01:10
@iCharlesHu

Copy link
Copy Markdown
Contributor

Hmm maybe this is why I can't get the docs to show up on swiftpackageindex.com... Thanks!!

@iCharlesHu iCharlesHu merged commit 02ba33c into main Jul 14, 2026
45 checks passed
@ktoso

ktoso commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Yeah that could be it tbh, we can ask Sven!

@ktoso ktoso deleted the ktoso-patch-1 branch July 14, 2026 02:10
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.

2 participants