Enforce Roslynator checks in CI build

[mregen]

• The ci pass includes Roslynator checks, fails on warnings
• Fix all existing warnings

[lgtm-com]

This pull request fixes 2 alerts when merging 24e0b55 into 486e97b - view on LGTM.com

fixed alerts:

• 2 for Constant condition

[codecov]

Codecov Report

Merging #1608 (24e0b55) into master (31e4a2b) will increase coverage by 0.07%.
The diff coverage is 3.11%.

@@            Coverage Diff             @@
##           master    #1608      +/-   ##
==========================================
+ Coverage   53.41%   53.49%   +0.07%     
==========================================
  Files         312      312              
  Lines       57364    57364              
==========================================
+ Hits        30640    30685      +45     
+ Misses      26724    26679      -45     

Impacted Files	Coverage Δ
....PubSub/Configuration/ConfigurationVersionUtils.cs	5.26% <ø> (ø)
...aries/Opc.Ua.PubSub/Transport/MqttClientCreator.cs	0.00% <0.00%> (ø)
Libraries/Opc.Ua.PubSub/Transport/UdpDiscovery.cs	0.00% <0.00%> (ø)
...s/Opc.Ua.PubSub/Transport/UdpDiscoveryPublisher.cs	0.00% <0.00%> (ø)
.../Opc.Ua.PubSub/Transport/UdpDiscoverySubscriber.cs	0.00% <0.00%> (ø)
...ies/Opc.Ua.PubSub/Transport/UdpPubSubConnection.cs	30.00% <0.00%> (ø)
Stack/Opc.Ua.Core/Stack/Nodes/ContentFilter.cs	5.27% <0.00%> (ø)
.../Opc.Ua.Core/Stack/Types/ContentFilter.Evaluate.cs	0.00% <0.00%> (ø)
...raries/Opc.Ua.PubSub/Transport/UdpClientCreator.cs	7.77% <33.33%> (+0.16%)	⬆️
...urity.Certificates/X509Certificate/X509PfxUtils.cs	69.87% <75.00%> (-0.50%)	⬇️
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31e4a2b...24e0b55. Read the comment docs.