Skip to content

test: implement Default for LimitedVec - #186

Open
0rlych1kk4 wants to merge 1 commit into
nasa:mainfrom
0rlych1kk4:fix/limitedvec-default
Open

0rlych1kk4 wants to merge 1 commit into
nasa:mainfrom
0rlych1kk4:fix/limitedvec-default

Conversation

@0rlych1kk4

Copy link
Copy Markdown

Summary

Implements Default for LimitedVec<T> in tests/util/inspector.rs by delegating to Self::new().

This resolves the clippy::new_without_default failure when running:

cargo clippy --test core_integration -- -D warnings

Testing

Verified locally with:

cargo clippy --test core_integration -- -D warnings

AI disclosure

ChatGPT was used to assist with drafting this pull request description. The submitted test-utility change was reviewed and verified locally.

@github-actions

Copy link
Copy Markdown

Welcome, new contributor!

Please make sure you've read our contributing guide, as well as our policy regarding AI usage, and we look forward to reviewing your pull request shortly

Signed-off-by: 0rlych1kk4 <orlychikka@gmail.com>
@0rlych1kk4
0rlych1kk4 force-pushed the fix/limitedvec-default branch from 5ff6fca to f152f3b Compare September 27, 2026 14:45
@0rlych1kk4

Copy link
Copy Markdown
Author

Hi @myint @kokroo @h313 could you please take a look when you have a chance? The branch is rebased onto the latest main and the workflow is awaiting maintainer approval.

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.

1 participant