Skip to content

[checkstyle] Enable check style for all the components#10544

Merged
xiangfu0 merged 1 commit intoapache:masterfrom
xiangfu0:fix-checkstyle-violations
Apr 5, 2023
Merged

[checkstyle] Enable check style for all the components#10544
xiangfu0 merged 1 commit intoapache:masterfrom
xiangfu0:fix-checkstyle-violations

Conversation

@xiangfu0
Copy link
Contributor

@xiangfu0 xiangfu0 commented Apr 4, 2023

Enable check style for all the modules

@xiangfu0 xiangfu0 force-pushed the fix-checkstyle-violations branch from dd7957c to 615a9a7 Compare April 4, 2023 23:40
@codecov-commenter
Copy link

codecov-commenter commented Apr 5, 2023

Codecov Report

Merging #10544 (615a9a7) into master (1fc8150) will increase coverage by 42.51%.
The diff coverage is 31.81%.

@@              Coverage Diff              @@
##             master   #10544       +/-   ##
=============================================
+ Coverage     27.80%   70.31%   +42.51%     
- Complexity       58     6464     +6406     
=============================================
  Files          2087     2103       +16     
  Lines        112307   112759      +452     
  Branches      16918    16977       +59     
=============================================
+ Hits          31226    79290    +48064     
+ Misses        77988    27922    -50066     
- Partials       3093     5547     +2454     
Flag Coverage Δ
integration1 24.48% <ø> (+0.13%) ⬆️
integration2 24.22% <ø> (+0.01%) ⬆️
unittests1 67.84% <ø> (?)
unittests2 13.87% <31.81%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...nnector/spark/common/CaseInsensitiveStringMap.java 35.41% <31.81%> (ø)

... and 1482 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@xiangfu0 xiangfu0 merged commit 764d28a into apache:master Apr 5, 2023
@xiangfu0 xiangfu0 deleted the fix-checkstyle-violations branch April 5, 2023 16:11
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.

3 participants