Skip to content

Conversation

@Itxaka
Copy link
Member

@Itxaka Itxaka commented Nov 13, 2025

  • Changed the method of flushing filesystem buffers from using exec.Command("sync") to syscall.Sync().
  • This change improves performance and reliability by directly invoking the system call instead of spawning a new process.

- Changed the method of flushing filesystem buffers from using `exec.Command("sync")` to `syscall.Sync()`.
- This change improves performance and reliability by directly invoking the system call instead of spawning a new process.

Signed-off-by: Itxaka <[email protected]>
@Itxaka Itxaka requested a review from a team as a code owner November 13, 2025 08:56
@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 41.43%. Comparing base (51d19f2) to head (90c05b0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/agent/hooks/encrypt.go 0.00% 1 Missing ⚠️

❌ Your project status has failed because the head coverage (59.27%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1035      +/-   ##
==========================================
- Coverage   41.48%   41.43%   -0.06%     
==========================================
  Files          62       62              
  Lines        6363     6362       -1     
==========================================
- Hits         2640     2636       -4     
- Misses       3398     3399       +1     
- Partials      325      327       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Itxaka Itxaka enabled auto-merge (squash) November 13, 2025 09:05
@Itxaka Itxaka merged commit 5309612 into main Nov 13, 2025
14 of 15 checks passed
@Itxaka Itxaka deleted the not_sync branch November 13, 2025 15:29
@Itxaka Itxaka mentioned this pull request Dec 9, 2025
32 tasks
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.

3 participants