Skip to content

Simplify Properties.toString implementation#1113

Open
twyatt wants to merge 2 commits intomainfrom
twyatt/properties-toString
Open

Simplify Properties.toString implementation#1113
twyatt wants to merge 2 commits intomainfrom
twyatt/properties-toString

Conversation

@twyatt
Copy link
Copy Markdown
Member

@twyatt twyatt commented Mar 5, 2026

Note

Low Risk
Simple refactoring using standard library function with no behavioral changes.

Overview
Refactors Characteristic.Properties.toString() to use Kotlin's buildList function instead of mutableListOf().apply { }, making the code more idiomatic and concise.

Reviewed by Cursor Bugbot for commit a066e61. Configure here.

@twyatt twyatt marked this pull request as ready for review April 22, 2026 20:57
@twyatt twyatt requested a review from a team as a code owner April 22, 2026 20:57
@twyatt twyatt added the patch Changes that should bump the PATCH version number label Apr 22, 2026
@twyatt twyatt enabled auto-merge (squash) April 23, 2026 22:12
@twyatt twyatt added the skip-changelog Excludes issue/PR from generated release change log. label Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch Changes that should bump the PATCH version number skip-changelog Excludes issue/PR from generated release change log.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants