fix: Adjust argument index for Fprintf in fmtpercentv linter check - #4443
Conversation
Fprintf in fmtpercentv linter checksFprintf in fmtpercentv linter check
gmlewis
left a comment
There was a problem hiding this comment.
Thank you, @Not-Dhananjay-Mishra!
LGTM.
Merging.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4443 +/- ##
=======================================
Coverage 97.55% 97.55%
=======================================
Files 194 194
Lines 19892 19892
=======================================
Hits 19406 19406
Misses 268 268
Partials 218 218 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Thanks @gmlewis Btw, I recently completed one year contributing here. I have learned a lot along the way. ❤️ |
Awesome! Congratulations, @Not-Dhananjay-Mishra and thank you very much for your contributions! |
Fprintfhas the following signaturePreviously, the linter always assumed the format string was at
expr.Args[0]#4436 (comment)