Skip to content

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

Merged
ermshiperete merged 1 commit into
epic/web-corefrom
feat/web/getKbdDetailsType
Apr 28, 2026
Merged

feat(web): clarify return type of KeymanEngine.getKeyboardDetails 🧼 🎼#15890
ermshiperete merged 1 commit into
epic/web-corefrom
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

Base automatically changed from feat/web/keyboardDetails to epic/web-core April 28, 2026 18:10
@ermshiperete ermshiperete merged commit 7c460e9 into epic/web-core Apr 28, 2026
8 of 9 checks passed
@ermshiperete ermshiperete deleted the feat/web/getKbdDetailsType branch April 28, 2026 18:11
@github-project-automation github-project-automation Bot moved this from Todo to Done in Keyman Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants