Skip to content

feat(developer): reviewRules: return object instead of string[] 😎#16073

Open
SabineSIL wants to merge 9 commits into
feat/developer/kmc-convertfrom
feat/developer/kmc-convert-warningMessages
Open

feat(developer): reviewRules: return object instead of string[] 😎#16073
SabineSIL wants to merge 9 commits into
feat/developer/kmc-convertfrom
feat/developer/kmc-convert-warningMessages

Conversation

@SabineSIL

Copy link
Copy Markdown
Contributor

during review of PR12564 we decided to refactor ReviewRules() so that it returns an interface instead of an string[]

see comment here

@keymanapp-test-bot skip

see also:

@keymanapp-test-bot

keymanapp-test-bot Bot commented Jun 7, 2026

Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot Bot changed the title feat(developer): reviewRules: return object instead of string[] messa… feat(developer): reviewRules: return object instead of string[] messa… 😎 Jun 7, 2026
@SabineSIL SabineSIL changed the title feat(developer): reviewRules: return object instead of string[] messa… 😎 feat(developer): reviewRules: return object instead of string[] 😎 Jun 24, 2026
@SabineSIL SabineSIL marked this pull request as ready for review June 26, 2026 08:03
@SabineSIL SabineSIL requested a review from mcdurdin June 26, 2026 08:27

@mcdurdin mcdurdin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@keyman-server keyman-server modified the milestones: A19S32, A19S33 Jul 4, 2026
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.

3 participants