chore: improve parameter naming of keys.ts

[ST-DDT]

Preparation for #2439

• Check eslint-plugin-unicorn rules to enable/configure/disable #2439

Related:

• chore: align coding style of deprecated.ts #3312
• chore: align coding style of lorem module #3313
• chore: align variable naming style of internet user agent #3314
• chore: improve variable naming of system module #3315
• chore: improve variable naming of helpers module #3316
• chore: improve parameter naming of internet module #3317
• chore: improve parameter naming of keys.ts #3318
• chore: align coding style of word module #3319

---

Improves the variable naming in the internet module.

---

This change is split from the no abbreviation lint PR, to reduce diff and I consider this slightly more valid than most of the other changes required for that.

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	ec6bf50
🔍 Latest deploy log	https://app.netlify.com/sites/fakerjs/deploys/674ee05e55fe6400087b6e8a
😎 Deploy Preview	https://deploy-preview-3318.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.96%. Comparing base (ecb5cb4) to head (ec6bf50).
Report is 3 commits behind head on next.

Additional details and impacted files

@@            Coverage Diff             @@
##             next    #3318      +/-   ##
==========================================
- Coverage   99.97%   99.96%   -0.01%     
==========================================
  Files        2806     2806              
  Lines      217137   217137              
  Branches      985      976       -9     
==========================================
- Hits       217076   217056      -20     
- Misses         61       81      +20     

Files with missing lines	Coverage Δ
src/internal/keys.ts	100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

[xDivisionByZerox]

I'm not sure about this change since I don't see any advantage in this parameter name change. Just like i is an acceptable abbreviated variable name for counting in itterations, obj is perfectly fine for arguments that represent a gerneric object IMO.

If we would like to completly activate the the prevent-abbreviations rule I'd propably not argue. But sice we are not, I don't see a reason to change this part of our code.