Skip to content

lexer: Always init line_ctr, for a more predictable interface, even if not needed#1567

Closed
erezsh wants to merge 1 commit intomasterfrom
issue1565
Closed

lexer: Always init line_ctr, for a more predictable interface, even if not needed#1567
erezsh wants to merge 1 commit intomasterfrom
issue1565

Conversation

@erezsh
Copy link
Copy Markdown
Member

@erezsh erezsh commented Oct 31, 2025

Attempts to solve issue #1565

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.97%. Comparing base (f79772c) to head (f04ec75).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1567   +/-   ##
=======================================
  Coverage   89.97%   89.97%           
=======================================
  Files          52       52           
  Lines        8018     8019    +1     
=======================================
+ Hits         7214     7215    +1     
  Misses        804      804           
Flag Coverage Δ
unittests 89.97% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@erezsh erezsh closed this Mar 26, 2026
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