OPCFoundation / UA-.NETStandard

OPC Unified Architecture .NET Standard
Other
1.9k stars 926 forks source link

Typos in names of functions. #2523

Closed mtx500 closed 4 months ago

mtx500 commented 4 months ago

Proposed changes

Fixed typos in names of functions. This may be a breaking change.

Related Issues

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply. You can also fill these out after creating the PR.

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Further comments

codecov[bot] commented 4 months ago

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (2b9d8f9) 53.81% compared to head (59fee7c) 53.79%.

Files Patch % Lines
...ies/Opc.Ua.Gds.Client.Common/CertificateWrapper.cs 0.00% 1 Missing :warning:
...s.Client.Common/RegisteredApplicationExtensions.cs 0.00% 1 Missing :warning:
Libraries/Opc.Ua.Server/Server/StandardServer.cs 0.00% 1 Missing :warning:
...ack/Opc.Ua.Core/Security/Certificates/X509Utils.cs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2523 +/- ## ========================================== - Coverage 53.81% 53.79% -0.03% ========================================== Files 334 334 Lines 64409 64409 Branches 13235 13235 ========================================== - Hits 34663 34649 -14 - Misses 25990 25999 +9 - Partials 3756 3761 +5 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mregen commented 4 months ago

Thanks for the update, due to the API change we need to wait for another breaking change / version bump.