Codecov Report

❌ Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.05%. Comparing base (8a0f652) to head (83d90ab).
⚠️ Report is 12 commits behind head on main.

@@            Coverage Diff             @@
##             main     #847      +/-   ##
==========================================
- Coverage   50.07%   50.05%   -0.03%     
==========================================
  Files         126      126              
  Lines       14420    14434      +14     
==========================================
+ Hits         7221     7225       +4     
- Misses       6718     6726       +8     
- Partials      481      483       +2     

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

Thanks @muhammad-asghar-ali! Do you mind undoing the changes that convert names from camel case to snake case? We use camel case pretty consistently so it would be nice to stick with that.

Thanks @muhammad-asghar-ali! Do you mind undoing the changes that convert names from camel case to snake case? We use camel case pretty consistently so it would be nice to stick with that.

Sure I'll do it... but I convert it b/c in go there is common approach to use snake_case naming instead of camelCase...

Sure I'll do it... but I convert it b/c in go there is common approach to use snake_case naming instead of camelCase...

I was curious so I looked it up, it looks like go recommends MixedCaps.

Sure I'll do it... but I convert it b/c in go there is common approach to use snake_case naming instead of camelCase...

I was curious so I looked it up, it looks like go recommends MixedCaps.

It's my personal observation that mostly go projects use the single word or snake_case variable naming. So finally the recommended way is MixedCaps, thanks for sharing... 😊

I'll revert the code as soon as possible

@dfarr changes done please check

Thank once again @muhammad-asghar-ali !!!