aliyun / aliyun-openapi-net-sdk

Alibaba Cloud SDK for .NET
https://www.nuget.org/profiles/aliyun-openapi-sdk
Other
541 stars 625 forks source link

Dispose IDisposables to prevent protential memory leak #204

Closed hanabi1224 closed 5 years ago

hanabi1224 commented 5 years ago

Fix warnings from https://www.nuget.org/packages/IDisposableAnalyzers/

codecov-io commented 5 years ago

Codecov Report

Merging #204 into master will increase coverage by 0.01%. The diff coverage is 98.83%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #204      +/-   ##
==========================================
+ Coverage   95.82%   95.84%   +0.01%     
==========================================
  Files          62       62              
  Lines        3042     3054      +12     
  Branches      287      287              
==========================================
+ Hits         2915     2927      +12     
  Misses         83       83              
  Partials       44       44
Flag Coverage Δ
#unittest 95.84% <98.83%> (+0.01%) :arrow_up:
Impacted Files Coverage Δ
aliyun-net-sdk-core/Reader/JsonReader.cs 97.65% <ø> (ø) :arrow_up:
...un-net-sdk-core/Regions/InternalEndpointsParser.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/Reader/XmlReader.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/Auth/SHA256withRSASigner.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/Auth/HmacSHA1Signer.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/Utils/ParameterHelper.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/Http/HttpResponse.cs 86.36% <96%> (+0.1%) :arrow_up:

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 a60291c...50744ea. Read the comment docs.