lavanet / lava

Apache License 2.0

407 stars 206 forks source link

fix: CNS-fix-modify-cli #1647

Closed Yaroms closed 2 months ago

Yaroms commented 2 months ago

Description

Closes: #XXXX

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

[ ] read the contribution guide
[ ] included the correct type prefix in the PR title, you can find examples of the prefixes below:
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] targeted the main branch
[ ] provided a link to the relevant issue or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] included the necessary unit and integration tests
[ ] updated the relevant documentation or specification, including comments for documenting Go code
[ ] confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic, API design and naming, documentation is accurate, tests and test coverage

github-actions[bot] commented 2 months ago

Test Results

2 173 tests +125 2 173 :white_check_mark: +128 25m 13s :stopwatch: +53s 145 suites + 2 0 :zzz: ± 0 7 files + 1 0 :x: - 3

Results for commit 0d98df3b. ± Comparison against base commit ea235a13.

:recycle: This comment has been updated with latest results.