Skip to content

Conversation

0marperez
Copy link
Contributor

Issue #

N/A

Description of changes

Enables custom ktlint rules from repo tools

See: aws/aws-kotlin-repo-tools#130

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

@0marperez 0marperez added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Sep 11, 2025

This comment has been minimized.

1 similar comment

This comment has been minimized.

Comment on lines +26 to +32
maven {
name = "kotlinRepoTools"
url = java.net.URI("https://d2gys1nrxnjnyg.cloudfront.net/releases")
content {
includeGroupByRegex("""aws\.sdk\.kotlin.*""")
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this needed for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's needed to resolve this dependency

@0marperez 0marperez marked this pull request as ready for review September 11, 2025 17:50
@0marperez 0marperez requested a review from a team as a code owner September 11, 2025 17:50
Copy link

Affected Artifacts

Changed in size
Artifact Pull Request (bytes) Latest Release (bytes) Delta (bytes) Delta (percentage)
runtime-core-jvm.jar 834,767 834,754 13 0.00%
http-client-jvm.jar 347,817 347,830 -13 -0.00%

@0marperez 0marperez merged commit f557442 into main Sep 16, 2025
20 checks passed
@0marperez 0marperez deleted the ktlint-custom-smithy-kotlin branch September 16, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants