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

Code format and refactor #210

Closed doublnt closed 5 years ago

doublnt commented 5 years ago
You need to complete
codecov-io commented 5 years ago

Codecov Report

Merging #210 into master will decrease coverage by 0.18%. The diff coverage is 96.52%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage   95.84%   95.66%   -0.19%     
==========================================
  Files          62       62              
  Lines        3058     2973      -85     
  Branches      288      279       -9     
==========================================
- Hits         2931     2844      -87     
+ Misses         83       82       -1     
- Partials       44       47       +3
Flag Coverage Δ
#unittest 95.66% <96.52%> (-0.19%) :arrow_down:
Impacted Files Coverage Δ
aliyun-net-sdk-core/Auth/HmacSHA1Signer.cs 100% <ø> (ø) :arrow_up:
aliyun-net-sdk-core/Auth/RoaSignatureComposer.cs 100% <ø> (ø) :arrow_up:
aliyun-net-sdk-core/Auth/SHA256withRSASigner.cs 100% <ø> (ø) :arrow_up:
...et-sdk-core/Auth/Sts/GetSessionAccessKeyRequest.cs 100% <ø> (ø) :arrow_up:
aliyun-net-sdk-core/Auth/Sts/AssumeRoleRequest.cs 100% <ø> (ø) :arrow_up:
aliyun-net-sdk-core/Auth/RpcSignatureComposer.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/CommonRoaRequest.cs 100% <100%> (ø) :arrow_up:
...t-sdk-core/Auth/Credential/EcsRamRoleCredential.cs 100% <100%> (ø) :arrow_up:
...et-sdk-core/Auth/Credential/AccessKeyCredential.cs 100% <100%> (ø) :arrow_up:
aliyun-net-sdk-core/CommonRequest.cs 100% <100%> (ø) :arrow_up:
... and 32 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 1e4d4b1...757a267. Read the comment docs.

AxiosLeo commented 5 years ago

是用代码修复工具批量修改的吗

doublnt commented 5 years ago

是用代码修复工具批量修改的吗

有个代码质量辅助的工具,可以查看警告,更改的。

AxiosLeo commented 5 years ago

是用代码修复工具批量修改的吗

有个代码质量辅助的工具,可以查看警告,更改的。

修改的有点多额。。。 这样review起来会比较有难度的

doublnt commented 5 years ago

是用代码修复工具批量修改的吗

有个代码质量辅助的工具,可以查看警告,更改的。

修改的有点多额。。。 这样review起来会比较有难度的

主要是改了一些命名的简写和一些代码格式化上的改进

AxiosLeo commented 5 years ago

是用代码修复工具批量修改的吗

有个代码质量辅助的工具,可以查看警告,更改的。

修改的有点多额。。。 这样review起来会比较有难度的

主要是改了一些命名的简写和一些代码格式化上的改进

好的,那也要都看一下的

doublnt commented 5 years ago

是用代码修复工具批量修改的吗

有个代码质量辅助的工具,可以查看警告,更改的。

修改的有点多额。。。 这样review起来会比较有难度的

主要是改了一些命名的简写和一些代码格式化上的改进

好的,那也要都看一下的

嗯呢, 我已经review了一遍, 还需要麻烦你再review一下, 可能哪些地方改动的有问题,你和我说一下。

AxiosLeo commented 5 years ago

嗯呢, 我已经review了一遍, 还需要麻烦你再review一下, 可能哪些地方改动的有问题,你和我说一下。

approve