Skip to content

add typotter to kotlin-sdk approver#541

Merged
nicklasl merged 1 commit intomainfrom
nicklasl-patch-1
Mar 27, 2026
Merged

add typotter to kotlin-sdk approver#541
nicklasl merged 1 commit intomainfrom
nicklasl-patch-1

Conversation

@nicklasl
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Nicklas Lundin <nicklasl@spotify.com>
@nicklasl nicklasl requested review from a team as code owners March 27, 2026 07:32
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new approver to the Kotlin SDK workgroup configuration. A review comment suggests that the added username 'typotter' might be a typo for 'staceypotter', as 'typotter' is not found in the organization's member list.


approvers:
- bencehornak
- typotter
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This username appears to be a typo for staceypotter, who is an existing organization member. To ensure permissions are assigned correctly, approvers should be existing members of the organization, as is the case with other members of this workgroup. The username typotter is not found in the member list in config/open-feature/org.yaml.

  - staceypotter

@nicklasl nicklasl merged commit 7f6e7e6 into main Mar 27, 2026
2 checks passed
@nicklasl nicklasl deleted the nicklasl-patch-1 branch March 27, 2026 13:53
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.

3 participants