Skip to content

fix: add .gitattributes to prevent binary corruption#8

Merged
calvinbrewer merged 3 commits into
mainfrom
fix/add-gitattributes-for-binary-files
Jun 27, 2025
Merged

fix: add .gitattributes to prevent binary corruption#8
calvinbrewer merged 3 commits into
mainfrom
fix/add-gitattributes-for-binary-files

Conversation

@coreyhn
Copy link
Copy Markdown
Collaborator

@coreyhn coreyhn commented Jun 27, 2025

Add .gitattributes file to mark platform-specific native libraries as binary files, preventing git from applying text transformations that corrupt the binaries during CI commits.

Update once_cell, serde, and serde_json to match resolved versions (primarily doing this to have the native libraries be rebuilt).

coreyhn added 2 commits June 27, 2025 11:00
Add `.gitattributes` file to mark platform-specific native libraries
as binary files, preventing git from applying text transformations
that corrupt the binaries during CI commits.
Update `once_cell`, `serde`, and `serde_json` to match resolved versions.
@calvinbrewer calvinbrewer merged commit 144c52b into main Jun 27, 2025
@calvinbrewer calvinbrewer deleted the fix/add-gitattributes-for-binary-files branch June 27, 2025 22:25
coreyhn added a commit that referenced this pull request Jun 27, 2025
Add platform native library files to Rust change detection. This allows
force rebuilds by deleting libraries when a clean rebuild of all
platforms is needed. This approach integrates cleanly with the existing
PR workflow without requiring manual workflow triggers or version bumps.

The library commit in PR #8 failed to update all platforms, maybe due
to a transient issue with the commit action. A validation step may be
needed if this rebuild resolves the issue.
coreyhn added a commit that referenced this pull request Jun 28, 2025
Remove `.gitattributes` file (from PR #8) which is no longer needed since
the GitHub API bypasses git's text transformation logic.
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