Skip to content

Conversation

@kul3r4
Copy link
Contributor

@kul3r4 kul3r4 commented Dec 8, 2025

Readd imports that were lost in previous PRs to show the imports that are useful in the Wear M3 migration guidance.

@kul3r4 kul3r4 marked this pull request as ready for review December 8, 2025 22:05
@kul3r4 kul3r4 requested a review from a team as a code owner December 8, 2025 22:05
@snippet-bot
Copy link

snippet-bot bot commented Dec 8, 2025

Here is the summary of changes.

You are about to add 4 region tags.
You are about to delete 4 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

@kul3r4 kul3r4 removed the request for review from ithinkihaveacat December 9, 2025 13:37
import androidx.compose.runtime.Composable
import androidx.compose.ui.platform.LocalContext
// [START android_wear_color]
import androidx.wear.compose.material3.ColorScheme
Copy link
Contributor

Choose a reason for hiding this comment

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

Being able to see the exact import is a good benefit of snippets, but I wonder if they can just be left in the GH source, rather than being visible on DAC. Among other things, you can only do this for the first snippet in the file (because of the way the EXCLUDE mechanism works).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

agree this approach has limitations and I have been trying to find other ways without much success. This case for showing imports is only for the migration guidance where we wanted to show the imports for clarity but unfortunately imports cannot be duplicated in the same file, so the only alternative is to create a new file for each one

@kul3r4 kul3r4 merged commit aea6b59 into main Dec 9, 2025
5 checks passed
@kul3r4 kul3r4 deleted the fix-imports branch December 9, 2025 13:54
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