Use ts strict checking for binding modbus

[relu91]

Yet another down.

Relates to #758.

[danielpeintner]

Looks good in general.

Have some understanding problems w.r.t. some error messages. See below

[codecov-commenter]

Codecov Report

Merging #1050 (dbf9979) into master (db4b2c1) will decrease coverage by 0.15%.
Report is 3 commits behind head on master.
The diff coverage is 67.85%.

@@            Coverage Diff             @@
##           master    #1050      +/-   ##
==========================================
- Coverage   75.63%   75.49%   -0.15%     
==========================================
  Files          80       80              
  Lines       15324    15385      +61     
  Branches     1460     1469       +9     
==========================================
+ Hits        11591    11615      +24     
- Misses       3699     3735      +36     
- Partials       34       35       +1     

Files Changed	Coverage Δ
...ckages/binding-modbus/src/modbus-client-factory.ts	40.00% <0.00%> (-0.82%)	⬇️
packages/binding-modbus/src/modbus-connection.ts	83.20% <63.63%> (-2.08%)	⬇️
packages/binding-modbus/src/modbus-client.ts	86.77% <84.61%> (-0.27%)	⬇️

... and 3 files with indirect coverage changes

[danielpeintner]

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

I tried already several times to activate the GitHub integration but I do not know "what" blocks it?

Shall I open a ticket on Eclipse side?

[danielpeintner]

Shall I open a ticket on Eclipse side?

FYI: I opened https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/3545

EDIT: Has been approved already .. cool. Let's see how the next PRs/commits look like

[relu91]

@danielpeintner do you think we can merge it even if we have redcrosses in codecov?

[danielpeintner]

@danielpeintner do you think we can merge it even if we have redcrosses in codecov?

We are now seeing these kind of "errors" in all our PRs now. I think we need to properly configure it and maybe show it as a warning or so.

Anyhow, w.r.t. this PR I think we can merge as is