-
Notifications
You must be signed in to change notification settings - Fork 39
AUTO : Forward from cleaning_3 to alpha #1605
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Relocate benchkit package to core modules directory - Update workspace configuration path reference - Change repository URLs to reflect new core location - Maintain all existing functionality and structure
- Standardize attribute formatting with proper spacing and grouping - Fix conditional compilation attributes for no_std feature - Remove redundant feature gates and optimize extern crate declarations - Apply consistent spacing patterns to type annotations and generics - Simplify strs_tools_meta macro parameters by removing unused use_simd option - Update optimize_split implementation with cleaner parameter handling
- Replace format string placeholders with inline formatting syntax - Change std:: imports to core:: for no_std compatibility - Add missing string imports for alloc compatibility - Update test ignore attribute with descriptive reason - Add comprehensive test script for crate-level validation
Resolved conflicts between assert_eq! approach and a_id! placeholder macro approach. Chose assert_eq! to maintain functional test validation over no-op placeholders.
- Remove unused std::collections::HashMap imports across modules - Update reflect_tools to use collection_tools HashMap consistently - Add HashMap<String, V> Fields implementation for &str key queries - Remove duplicate Cells implementation for std HashMap - Update test imports to use collection_tools HashMap - Clean up debug mode parameters in optimization test files - Update time_tools readme with proper feature conditionals - Standardize collection imports across format_tools and reflect_tools
- Replace Default::default() with explicit struct constructors - Add reference parameters to prevent unnecessary moves - Update ignore attributes with descriptive reasons - Fix clippy warnings for unused variables and float comparisons - Standardize numeric literal formatting with underscores - Remove redundant all() wrappers in cfg conditions - Update core imports to prefer core over std where applicable
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an automated pull request to update from branch cleaning_3