Skip to content

Conversation

murkitsa
Copy link
Contributor

@murkitsa murkitsa commented Dec 5, 2024

CUB-5407

Description

Manual PR Checklist

  • All related Jira tickets added to PR title
  • Description in Jira ticket is clear and up-to-date

@ignatiusreza ignatiusreza added the pushback PR comments needs to be addressed or discussed label Dec 5, 2024
@murkitsa murkitsa removed the pushback PR comments needs to be addressed or discussed label Dec 6, 2024
@ignatiusreza ignatiusreza added the pushback PR comments needs to be addressed or discussed label Dec 8, 2024
@murkitsa murkitsa removed the pushback PR comments needs to be addressed or discussed label Dec 10, 2024
Copy link
Contributor

@ignatiusreza ignatiusreza left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Review checklist according to SOP-13 & SOP-014

  • Code is readable
  • Test coverage is appropriate
  • I manually tested the changes and checked that related capabilities do not have any regressions
  • I confirmed that all specifications linked to the issue are fulfilled
  • I checked newly added dependencies for their licenses

@kgrhartlage kgrhartlage added the automerge To be merged automatically by Kodiak label Dec 10, 2024
@labforward-kodiak labforward-kodiak bot merged commit 81bf575 into main Dec 10, 2024
4 checks passed
@labforward-kodiak labforward-kodiak bot deleted the task/CUB-5407-sendCommand-handlers branch December 10, 2024 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge To be merged automatically by Kodiak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants