Skip to content

refactor: replace short hand type conversions with function calls#11

Merged
jerrythomas merged 2 commits intodevelopfrom
deepsource-autofix-001ffb9c
Mar 20, 2024
Merged

refactor: replace short hand type conversions with function calls#11
jerrythomas merged 2 commits intodevelopfrom
deepsource-autofix-001ffb9c

Conversation

@deepsource-autofix
Copy link
Contributor

Prefer using explicit casts by calling Number, Boolean, or String over using operators like +, !! or "" +. This is considered best practice as it improves readability.

Prefer using explicit casts by calling `Number`, `Boolean`, or `String` over using operators like `+`, `!!` or `"" +`. This is considered best practice as it improves readability.
@vercel
Copy link

vercel bot commented Mar 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rokkit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 20, 2024 4:31pm

This commit fixes the style issues introduced in 83a8c63 according to the output
from Prettier.

Details: #11
@deepsource-io
Copy link
Contributor

deepsource-io bot commented Mar 20, 2024

Here's the code health analysis summary for commits b548448..a4e9483. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Test coverage LogoTest coverage⚠️ Artifact not reportedTimed out: Artifact was never reportedView Check ↗
DeepSource JavaScript LogoJavaScript✅ Success
🎯 12 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@jerrythomas jerrythomas merged commit 9dc2f07 into develop Mar 20, 2024
@jerrythomas jerrythomas deleted the deepsource-autofix-001ffb9c branch March 20, 2024 16:32
jerrythomas pushed a commit that referenced this pull request May 6, 2025
This commit fixes the style issues introduced in b08885f according to the output
from Prettier.

Details: #11
jerrythomas added a commit that referenced this pull request May 6, 2025
refactor: replace short hand type conversions with function calls
jerrythomas added a commit that referenced this pull request Feb 26, 2026
…11, #28, #8)

- Type-ahead search for List/Tree: findByText() on ListController + typeahead option on navigator action with 500ms buffer reset and wrapping search
- MultiSelect value contract aligned with Select/List/Tree: value is now unknown[] (primitives), added selected bindable (SelectItem[]), onchange emits (values, items)
- Legacy cleanup: deleted broken ListEditor/NestedEditor/DataEditor, stale snapshots
- Marked backlog #25 (bits-ui) and #58 (Svelte 4→5) as done
- Added kbd.js optimization backlog item (#6)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.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.

1 participant