Skip to content

Conversation

@cartland
Copy link
Contributor

The lint tool reported ComposableNaming warnings in GlanceSnippets.kt. Composable functions that return Unit should be named using PascalCase, like classes, to clearly identify them as UI components.

This commit renames 12 Composable functions to adhere to this convention. For example, 'actionLambda' has been renamed to 'ActionLambda'. This follows Compose best practices.

The lint tool reported "ComposableNaming" warnings in GlanceSnippets.kt.
Composable functions that return Unit should be named using PascalCase,
like classes, to clearly identify them as UI components.

This commit renames 12 Composable functions to adhere to this convention.
For example, `actionLambda` has been renamed to `ActionLambda`.
This improves code readability and follows Compose best practices.
@cartland cartland added the enhancement New feature or request label Aug 29, 2025
@cartland cartland merged commit d6d74ed into main Aug 29, 2025
4 checks passed
@cartland cartland deleted the cartland/refactor-composable-naming branch August 29, 2025 21:21
dturner added a commit that referenced this pull request Sep 23, 2025
* main: (74 commits)
  Add basic WindowInsetsRulers cases (#621)
  Adding shadows code snippets (#620)
  Update the wear preview code
  Update state based (#619)
  add WindowInsetsRulers snippet (#616)
  Migrate snippets from dac/training/wearables/tiles/screen-size (#617)
  Resolve warnings in the Wakelock code snippets (#615)
  Disabling some "unused" warnings (#610)
  Add android_wear_tile_version_fallback (#614)
  Add viewmodel screen for KMP (#613)
  Fix missing snippet (#612)
  Add KMP ViewModel snippets (#611)
  docs(modifiers): Add TODOs for inspectableProperties lint warnings (#589)
  Fix: Prevent memory leak from implicit SAM conversion (#588)
  Fix(gestures): Handle pointer events inside awaitPointerEventScope (#590)
  Lint: Use specialized state holders to avoid autoboxing (#591)
  Refactor: Rename Composable functions to follow naming conventions (#592)
  Refactor(text): Use KTX extension for isDigitsOnly check (#593)
  Refactor(SearchBar): Reorder modifier parameter to follow convention (#594)
  Add padding to avoid overwriting TimeText (#529)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants