Skip to content

Fix build issues and clean up temporary files #188

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 1 commit into from
Jun 13, 2025
Merged

Conversation

gmorpheme
Copy link
Member

Summary

  • Fix critical compilation issue with Block::fill method
  • Remove accidentally committed temporary files
  • Update .gitignore to prevent future temporary file commits

Changes Made

  • Fixed Block::fill method: Moved #[cfg(debug_assertions)] inside method body to make method always available while keeping debug-only behavior
  • Removed temporary files: Cleaned up .DS_Store, .cache/, and test.eu files that were accidentally committed
  • Updated .gitignore: Added patterns to prevent future commits of temporary files

Build Status

  • cargo build - Compiles successfully
  • cargo clippy --lib -- -D warnings - No lint warnings
  • ✅ Addresses benchmark build failures from previous merge

Testing

  • Library tests pass (160/161 - one unrelated GC test failure)
  • Clean compilation in both debug and release modes
  • No clippy warnings

🤖 Generated with Claude Code

- Fix Block::fill method compilation issue by moving conditional compilation inside method body
- Remove accidentally committed temporary files (.DS_Store, .cache/, test.eu)
- Update .gitignore to prevent future commits of temporary files
- Ensures clean builds in both debug and release modes

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@gmorpheme gmorpheme merged commit b44e126 into master Jun 13, 2025
17 of 18 checks passed
@gmorpheme gmorpheme deleted the hotfix/build-fixes branch June 13, 2025 09:00
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