zls icon indicating copy to clipboard operation
zls copied to clipboard

Published 20 hours ago • verified publisher icon zigtools

Add semicolon when it's missing

Open SuperAuguste opened this issue 2 years ago • 2 comments

Closes #1475

Probably needs some fuzzing and manual testing but this is it!

I wonder if applying multiple autofixes at once would be desirable as now this can happen pretty easily (see var a = 1, which adds a ; on the first save and then an unused discard on the second.)

Anyways back to my job I go! 🏃‍♂️

SuperAuguste avatar Sep 27 '23 12:09 SuperAuguste

Is this something anyone actually wants? Not sure if it's useful

SuperAuguste avatar Dec 05 '23 06:12 SuperAuguste

If this feature has no false-positives then it wouldn't hurt, right?

Techatrix avatar Dec 05 '23 18:12 Techatrix

Codecov Report

Attention: Patch coverage is 20.00000% with 16 lines in your changes are missing coverage. Please review.

Project coverage is 78.57%. Comparing base (d034594) to head (0051998). Report is 1 commits behind head on master.

Files Patch % Lines
src/features/code_actions.zig 0.00% 8 Missing :warning:
src/features/diagnostics.zig 27.27% 8 Missing :warning:
Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1476      +/-   ##
==========================================
- Coverage   78.69%   78.57%   -0.13%     
==========================================
  Files          34       34              
  Lines       10560    10584      +24     
==========================================
+ Hits         8310     8316       +6     
- Misses       2250     2268      +18

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Mar 03 '24 01:03 codecov[bot]