Skip to content

Conversation

@ZacSweers
Copy link
Collaborator

No description provided.

manifest { attributes("Automatic-Module-Name" to "com.squareup.moshi.adapters") }
}

configure<MavenPublishBaseExtension> { configure(KotlinJvm(javadocJar = None())) }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this prevent it from publishing? Or do we get some default behaviour? In Okio we do this:

configure<MavenPublishBaseExtension> {
  configure(
    KotlinMultiplatform(javadocJar = JavadocJar.Empty())
  )
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Applying the plugin alone is enough I think since we have this auto-configuring from the root project on subprojects. This line seemed to just disable javadoc jars, which is what was breaking sonatype's validation per @gabrielittner

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Worst case it still fails validation and we try something else 😅

Copy link
Contributor

Choose a reason for hiding this comment

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

When you apply the base plugin you need to call one of the configure methods from the DSL. There are cases where we just configure publications created by others then it might still work, in other cases nothing will get published at all

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hmmm I missed the "base" bit. I'm not sure why we have that tbh, stuck a PR up here to remove that and use the standard setup: #2077

@ZacSweers ZacSweers merged commit 8667092 into master Jan 28, 2026
6 of 7 checks passed
@ZacSweers ZacSweers deleted the z/fixReleasing branch January 28, 2026 21:15
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.

4 participants