Skip to content

fix(worker): Handle CRLF edits in snapshot creation#3837

Merged
nellh merged 1 commit intomasterfrom
newline-snapshot-fix
Mar 25, 2026
Merged

fix(worker): Handle CRLF edits in snapshot creation#3837
nellh merged 1 commit intomasterfrom
newline-snapshot-fix

Conversation

@nellh
Copy link
Copy Markdown
Contributor

@nellh nellh commented Mar 25, 2026

This fixes an issue with automated edits (usually during snapshot creation) where CRLF in the edited files could lead to a mismatch between the expected content and the actual file. Normalizing CRLF to LF in this check solves it.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.40%. Comparing base (3e69eac) to head (8d6b48c).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3837   +/-   ##
=======================================
  Coverage   45.40%   45.40%           
=======================================
  Files         650      650           
  Lines       34957    34957           
  Branches     1711     1711           
=======================================
  Hits        15873    15873           
  Misses      18902    18902           
  Partials      182      182           

☔ 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.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026

Bundle Report

Bundle size has no change ✅

@nellh nellh force-pushed the newline-snapshot-fix branch from ac30142 to 8d6b48c Compare March 25, 2026 16:18
@nellh nellh merged commit a133a58 into master Mar 25, 2026
18 checks passed
@nellh nellh deleted the newline-snapshot-fix branch March 25, 2026 16:22
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