Skip to content

feat: modernize Kotlin snippets module with capabilities and visual test suites - #2427

Open
dkhawk wants to merge 1 commit into
feat/snippets-java-appfrom
feat/snippets-kotlin-app
Open

dkhawk wants to merge 1 commit into
feat/snippets-java-appfrom
feat/snippets-kotlin-app

Conversation

@dkhawk

@dkhawk dkhawk commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Create :snippets:kotlin-app with 14 snippet categories, KTX extensions, and documentation region tags
  • Add Kotlin snippet infrastructure (KotlinSnippetsActivity, MapActivity, SnippetRegistry, TrackedMap)
  • Add Kotlin capabilities test suite (CatalogCapabilitiesTestSuite, CameraControl, Events, MapInit, Marker)
  • Add Kotlin visual test suite (BaseVisualTest, VisualTests)
  • Remove legacy Kotlin snippet modules (snippets/app-ktx, snippets/app-utils-ktx)
  • Update root settings.gradle.kts

Stacked Base

Stacked on #2426 (feat/snippets-java-app).

Reviewers

@kikoso

@dkhawk
dkhawk force-pushed the feat/snippets-kotlin-app branch from 3eb7aaa to 8c6b89b Compare September 15, 2026 00:25
@dkhawk
dkhawk force-pushed the feat/snippets-kotlin-app branch from 8c6b89b to 293149c Compare September 15, 2026 00:35
@dkhawk
dkhawk marked this pull request as ready for review September 15, 2026 00:38
@dkhawk
dkhawk requested a review from kikoso September 15, 2026 00:38
@snippet-bot

snippet-bot Bot commented Sep 15, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 128 region tags.
You are about to delete 40 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@dkhawk
dkhawk added this pull request to stack #2429 September 15, 2026 00:43
@dkhawk
dkhawk force-pushed the feat/snippets-kotlin-app branch from 293149c to 00469c2 Compare September 15, 2026 18:22
…est suites

- Create :snippets:kotlin-app with 14 snippet categories, KTX extensions, and documentation region tags
- Add Kotlin snippet infrastructure (KotlinSnippetsActivity, MapActivity, SnippetRegistry, TrackedMap)
- Add Kotlin capabilities test suite (CatalogCapabilitiesTestSuite, CameraControl, Events, MapInit, Marker)
- Add Kotlin visual test suite (BaseVisualTest, VisualTests)
- Remove legacy Kotlin snippet modules (snippets/app-ktx, snippets/app-utils-ktx)
- Update root settings.gradle.kts
@dkhawk
dkhawk force-pushed the feat/snippets-kotlin-app branch from 00469c2 to 37ae58f Compare September 15, 2026 22:54

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Android Lint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

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