Skip to content

feat(web): clarify return type of KeymanEngine.getKeyboardDetails 🧼 🎼#15890

Open
ermshiperete wants to merge 1 commit intofeat/web/keyboardDetailsfrom
feat/web/getKbdDetailsType
Open

feat(web): clarify return type of KeymanEngine.getKeyboardDetails 🧼 🎼#15890
ermshiperete wants to merge 1 commit intofeat/web/keyboardDetailsfrom
feat/web/getKbdDetailsType

Conversation

@ermshiperete
Copy link
Copy Markdown
Contributor

getKeyboardDetails can return null if stub is falsy. This PR changes the return type accordingly to improve type safety.

Follows: #15889
Build-bot: skip build:web
Test-bot: skip

`getKeyboardDetails` can return `null` if `stub` is falsy. This PR changes
the return type accordingly to improve type safety.

Follows: #15889
Build-bot: skip build:web
Test-bot: skip
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot Bot commented Apr 27, 2026

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

  • Web
    • KeymanWeb Test Home - build : all tests passed (no artifacts on BuildLevel "build")

@keymanapp-test-bot keymanapp-test-bot Bot changed the title feat(web): clarify return type of KeymanEngine.getKeyboardDetails 🧼 feat(web): clarify return type of KeymanEngine.getKeyboardDetails 🧼 🎼 Apr 27, 2026
@keymanapp-test-bot keymanapp-test-bot Bot added this to the A19S28 milestone Apr 27, 2026
Copy link
Copy Markdown
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants