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

Fix NPE in NewLineRule #792

Merged
merged 4 commits into from
Mar 9, 2021
Merged

Fix NPE in NewLineRule #792

merged 4 commits into from
Mar 9, 2021

Conversation

Cheshiriks
Copy link
Member

@Cheshiriks Cheshiriks commented Mar 5, 2021

What's done:

### What's done:
* Fixed NPE in NewLineRule for lambda list argument
@Cheshiriks Cheshiriks force-pushed the bugfix/npe-newline branch from b5592bd to f03adbd Compare March 5, 2021 14:57
@codecov
Copy link

codecov bot commented Mar 5, 2021

Codecov Report

Merging #792 (03958bc) into master (8891a13) will decrease coverage by 0.00%.
The diff coverage is 72.72%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #792      +/-   ##
============================================
- Coverage     80.92%   80.91%   -0.01%     
- Complexity     2203     2204       +1     
============================================
  Files            98       98              
  Lines          5666     5670       +4     
  Branches       1753     1756       +3     
============================================
+ Hits           4585     4588       +3     
  Misses          285      285              
- Partials        796      797       +1     
Flag Coverage Δ Complexity Δ
unittests 80.91% <72.72%> (-0.01%) 0.00 <13.00> (ø)

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

Impacted Files Coverage Δ Complexity Δ
...iktat/ruleset/rules/chapter3/files/NewlinesRule.kt 77.22% <72.72%> (-0.04%) 146.00 <13.00> (+1.00) ⬇️

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 8891a13...03958bc. Read the comment docs.

Copy link
Member

@orchestr7 orchestr7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add tests and fix null issues :)

### What's done:
* Fixed NPE in NewLineRule for lambda list argument
Copy link
Member

@petertrr petertrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just fix code style

### What's done:
* Fixed NPE in NewLineRule for lambda list argument
@Cheshiriks Cheshiriks force-pushed the bugfix/npe-newline branch from c98ed21 to 4fdb427 Compare March 9, 2021 10:47
Copy link
Member

@orchestr7 orchestr7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@petertrr petertrr merged commit 386ae91 into master Mar 9, 2021
@petertrr petertrr deleted the bugfix/npe-newline branch March 9, 2021 11:33
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.

NPE: Internal error has occurred in newlines
3 participants