Skip to content

misc: upgrade to Dokka 2.0.0 #1597

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

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

misc: upgrade to Dokka 2.0.0 #1597

wants to merge 25 commits into from

Conversation

lauzadis
Copy link
Member

Issue #

Description of changes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@lauzadis lauzadis changed the title misc: upgrade to Dokka 2.0 misc: upgrade to Dokka 2.0.0 May 15, 2025
Copy link

A new generated diff is ready to view.

@lauzadis lauzadis marked this pull request as ready for review May 15, 2025 21:45
@lauzadis lauzadis requested a review from a team as a code owner May 15, 2025 21:45
Copy link

A new generated diff is ready to view.

Copy link

A new generated diff is ready to view.

This comment has been minimized.

@lauzadis lauzadis added the ready-for-release Overrides failing "release-readiness" check, when you're sure this PR is ready for release. label May 15, 2025
Copy link

Copy link

A new generated diff is ready to view.

Copy link

Affected Artifacts

Changed in size
Artifact Pull Request (bytes) Latest Release (bytes) Delta (bytes) Delta (percentage)
outposts-jvm.jar closure 9,267,093 9,267,087 6 0.00%
workspaces-jvm.jar closure 11,491,795 11,502,501 -10,706 -0.09%
databasemigrationservice-jvm.jar closure 12,692,465 12,708,045 -15,580 -0.12%
codebuild-jvm.jar closure 10,729,796 10,755,661 -25,865 -0.24%
pcs-jvm.jar closure 8,789,652 8,814,272 -24,620 -0.28%
workspaces-jvm.jar 3,585,918 3,596,625 -10,707 -0.30%
databasemigrationservice-jvm.jar 4,786,588 4,802,169 -15,581 -0.32%
codebuild-jvm.jar 2,823,919 2,849,785 -25,866 -0.91%
bedrockagent-jvm.jar closure 12,799,178 13,091,229 -292,051 -2.23%
pcs-jvm.jar 883,775 908,396 -24,621 -2.71%
bedrockagent-jvm.jar 4,893,301 5,185,353 -292,052 -5.63%

@@ -90,7 +88,8 @@ tasks.test {
// FIXME Commonize the following functions into the aws-kotlin-repo-tools build-support
val sdkVersion: String by project

@OptIn(InternalApi::class)
private fun String.ensureSuffix(suffix: String): String = if (endsWith(suffix)) this else plus(suffix)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we no longer using import aws.smithy.kotlin.runtime.text.ensureSuffix? Can we de-duplicate this function using buildSrc?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-release Overrides failing "release-readiness" check, when you're sure this PR is ready for release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants