Skip to content

chore: fix mut schema builder public api#420

Merged
luoyuxia merged 2 commits into
apache:mainfrom
fresh-borzoni:chore-fix-mut-schema
Mar 3, 2026
Merged

chore: fix mut schema builder public api#420
luoyuxia merged 2 commits into
apache:mainfrom
fresh-borzoni:chore-fix-mut-schema

Conversation

@fresh-borzoni

Copy link
Copy Markdown
Member

Fix mut requirement for schema builder

@fresh-borzoni

fresh-borzoni commented Mar 3, 2026

Copy link
Copy Markdown
Member Author

@luoyuxia I guess with this we don't need users use mut
I checked, we can leave callers with mut references for reassignment for internal code. but for public API we shouldn't require users to use mutable schema

@fresh-borzoni

Copy link
Copy Markdown
Member Author

fixed clippy warning about not used mut

@luoyuxia luoyuxia merged commit 5718d20 into apache:main Mar 3, 2026
8 checks passed
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.

2 participants