Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

validates javadoc for every commit #338

Merged

Conversation

xiangyushawn
Copy link
Contributor

re-create PR for #327 to RTW branch

@codecov-io
Copy link

Codecov Report

Merging #338 into RTW_6.2.0 will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             RTW_6.2.0     #338      +/-   ##
===============================================
+ Coverage        37.16%   37.25%   +0.08%     
- Complexity        1669     1673       +4     
===============================================
  Files              103      103              
  Lines            23662    23662              
  Branches          3918     3918              
===============================================
+ Hits              8795     8815      +20     
+ Misses           13278    13253      -25     
- Partials          1589     1594       +5
Flag Coverage Δ Complexity Δ
#JDBC41 37.04% <ø> (+0.02%) 1659 <ø> (-1) ⬇️
#JDBC42 37.16% <ø> (+0.07%) 1670 <ø> (+3) ⬆️
Impacted Files Coverage Δ Complexity Δ
...om/microsoft/sqlserver/jdbc/ReaderInputStream.java 44.94% <0%> (ø) 16% <0%> (ø) ⬇️
...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java 49% <0%> (+0.21%) 211% <0%> (+1%) ⬆️
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java 46.19% <0%> (+0.23%) 0% <0%> (ø) ⬇️
...c/main/java/com/microsoft/sqlserver/jdbc/Util.java 47.95% <0%> (+0.68%) 61% <0%> (+1%) ⬆️
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 30.56% <0%> (+1.12%) 57% <0%> (+1%) ⬆️
...a/com/microsoft/sqlserver/jdbc/PLPInputStream.java 40.96% <0%> (+1.2%) 29% <0%> (+1%) ⬆️

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 1da02b5...9c6206a. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Jun 13, 2017

Codecov Report

Merging #338 into RTW_6.2.0 will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             RTW_6.2.0     #338      +/-   ##
===============================================
- Coverage        37.16%   37.12%   -0.05%     
+ Complexity        1669     1667       -2     
===============================================
  Files              103      103              
  Lines            23662    23662              
  Branches          3918     3918              
===============================================
- Hits              8795     8784      -11     
- Misses           13278    13290      +12     
+ Partials          1589     1588       -1
Flag Coverage Δ Complexity Δ
#JDBC41 37.05% <ø> (+0.02%) 1663 <ø> (+3) ⬆️
#JDBC42 36.92% <ø> (-0.17%) 1658 <ø> (-9)
Impacted Files Coverage Δ Complexity Δ
src/main/java/microsoft/sql/DateTimeOffset.java 37.14% <0%> (-2.86%) 8% <0%> (-2%)
...m/microsoft/sqlserver/jdbc/SQLServerException.java 75.6% <0%> (-1.63%) 28% <0%> (-1%)
...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java 36.72% <0%> (-0.28%) 0% <0%> (ø)
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 29.21% <0%> (-0.23%) 55% <0%> (-1%)
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java 45.78% <0%> (-0.18%) 0% <0%> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java 27.28% <0%> (+0.04%) 188% <0%> (+1%) ⬆️
...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java 48.86% <0%> (+0.07%) 210% <0%> (ø) ⬇️
...om/microsoft/sqlserver/jdbc/ReaderInputStream.java 46.06% <0%> (+1.12%) 17% <0%> (+1%) ⬆️

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 1da02b5...6aa193e. Read the comment docs.

@v-nisidh v-nisidh merged commit 17aee6f into microsoft:RTW_6.2.0 Jun 16, 2017
@xiangyushawn xiangyushawn deleted the validates-javadoc-by-default branch September 8, 2017 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants