Closed mergify[bot] closed 6 months ago
Cherry-pick of cf09bb59fa0ba3c5d96f3018dbd74131fccef57c has failed:
On branch mergify/bp/osmo-v23/v0.47.5-iavl-v1/pr-572
Your branch is up to date with 'origin/osmo-v23/v0.47.5-iavl-v1'.
You are currently cherry-picking commit cf09bb59f.
(fix conflicts and run "git cherry-pick --continue")
(use "git cherry-pick --skip" to skip this patch)
(use "git cherry-pick --abort" to cancel the cherry-pick operation)
Changes to be committed:
modified: server/util.go
Unmerged paths:
(use "git add <file>..." to mark resolution)
both modified: CHANGELOG.md
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally
[!IMPORTANT]
Auto Review Skipped
Bot user detected.
To trigger a single review, invoke the
@coderabbitai review
command.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description
Closes: #XXXX
There is some really strange bug with appOpts.Get for string slices. I spent hours trying to figure out the root cause, but was unable to. Basically, it sets the result as an array of length 1, no mater how it is formated, along with adding an extra set of square brackets around the result.
So ["lockup"] becomes [lockup], ["lockup", "superfluid"] becomes [lockup superfluid], etc.
This just adds separate logic to get around this problem.
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...
!
to the type prefix if API or client breaking changeCHANGELOG.md
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...
!
in the type prefix if API or client breaking changeSummary by CodeRabbit
Bug Fixes
New Features
This is an automatic backport of pull request #572 done by Mergify.