lizhimins commented on PR #6845: URL: https://github.com/apache/rocketmq-dashboard/pull/6845#issuecomment-6441474970
Merged as part of #6851 - squashed into `rocketmq-studio` as `da2513c9f`. Your change here (declaring `maximum: 100` for the three paged tools and carrying it end to end through rmqctl, matching the ceiling all three services already enforce. `catalog_gen.go` is regenerated and `make catalog-verify` reports no drift. For what it is worth, the constraint existed in `rmq-tools.yaml` before #4225 split the catalog and was silently dropped there, because the generator's allow-list did not include `maximum`.) is included. We grouped 11 of your related fixes into a single commit so the trunk history stays reviewable; the other PRs in the group are closed with the same note. Everything was cherry-picked onto the current baseline with you kept as the author, and your tests came along with the change - the combined branch runs 4143 backend tests and 2009 frontend tests green, with the catalog, license and lint gates clean. Where the note above says "we", that part was added while landing the batch; the rest is yours unchanged. Thanks for the consistently well-tested patches. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
