Skip to content

Comments

Switch AI module to m3#2647

Merged
yschimke merged 8 commits intogoogle:mainfrom
yschimke:m3_ai
Jul 10, 2025
Merged

Switch AI module to m3#2647
yschimke merged 8 commits intogoogle:mainfrom
yschimke:m3_ai

Conversation

@yschimke
Copy link
Collaborator

@yschimke yschimke commented Jul 9, 2025

WHAT

WHY

HOW

Checklist 📋

  • Add explicit visibility modifier and explicit return types for public declarations
  • Run spotless check
  • Run tests
  • Update metalava's signature text files

@yschimke yschimke requested a review from kul3r4 July 9, 2025 17:49
Copy link
Collaborator

@kul3r4 kul3r4 left a comment

Choose a reason for hiding this comment

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

great!
I left a couple of comments.

) {
Icon(
imageVector = Icons.Default.QuestionAnswer,
contentDescription = "Ask Again",
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: string resource for the future?

@yschimke
Copy link
Collaborator Author

@kul3r4 updated to use EdgeButton also

@yschimke yschimke merged commit 6c2d466 into google:main Jul 10, 2025
6 checks passed
kul3r4 pushed a commit to kul3r4/horologist-1 that referenced this pull request Sep 19, 2025
* Switch AI module to m3

---------

Co-authored-by: yschimke <231923+yschimke@users.noreply.github.com>
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