chore: turn on noImplicitThis

[ST-DDT]

Fixes #1835, #1838

• Turn on strictBindCallApply in tsconfig #1835
• Turn on noImplicitThis in tsconfig #1838

Both rules need to be changed at the same time, because they cannot be fixed individually.

[Shinigami92]

As already discussed in Discord, but to transparently ask this here again:
Does this still work with something like

class X {
  y = 1
  a() {
    return this.y
  }
}
const x = new X()
unqiue(x.a) // is y still bound to the class instance?

[ST-DDT]

In v7.6 and next it outputs undefined.
After this PR it will throw an error.

Neither is correct.

FFR: In v7.6 and next this is { exec: [Getter] } which isn't useful at all.

FWIW Both variants work when using faker.helpers.unique(x.a.bind(x)).

[codecov]

Codecov Report

Merging #1842 (4a2bcb7) into next (be5bd5c) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             next    #1842      +/-   ##
==========================================
- Coverage   99.63%   99.62%   -0.02%     
==========================================
  Files        2346     2346              
  Lines      236489   236489              
  Branches     1148     1141       -7     
==========================================
- Hits       235633   235600      -33     
- Misses        834      867      +33     
  Partials       22       22              

Impacted Files	Coverage Δ
src/modules/helpers/index.ts	99.21% <100.00%> (ø)
src/modules/helpers/unique.ts	98.78% <100.00%> (ø)
src/modules/internet/user-agent.ts	84.61% <0.00%> (-9.77%)	⬇️