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

Enable null checker on exception files #1021

Merged
merged 1 commit into from
Sep 28, 2020
Merged

Conversation

dahlia
Copy link
Contributor

@dahlia dahlia commented Sep 27, 2020

Enable null checker on exceptions files. This contributes to #458.

Total files:

$ find Libplanet/ -name '*.cs' | wc -l
187

Files that enable null checker (39%):

$ git grep '#nullable enable' Libplanet/ | wc -l
73

Note that it was before (19%):

$ git grep '#nullable enable' Libplanet/ | wc -l
37

@codecov
Copy link

codecov bot commented Sep 27, 2020

Codecov Report

Merging #1021 into main will decrease coverage by 1.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1021      +/-   ##
==========================================
- Coverage   88.45%   87.37%   -1.09%     
==========================================
  Files         331      300      -31     
  Lines       30203    28795    -1408     
==========================================
- Hits        26717    25159    -1558     
- Misses       1882     2051     +169     
+ Partials     1604     1585      -19     
Impacted Files Coverage Δ
Libplanet/Action/MissingActionTypeException.cs 80.00% <ø> (ø)
...et/Action/UnexpectedlyTerminatedActionException.cs 75.00% <ø> (-17.60%) ⬇️
...lanet/Blockchain/IncompleteBlockStatesException.cs 37.50% <ø> (ø)
...ibplanet/Blocks/InvalidBlockDifficultyException.cs 100.00% <ø> (ø)
Libplanet/Blocks/InvalidBlockException.cs 50.00% <ø> (ø)
Libplanet/Blocks/InvalidBlockHashException.cs 100.00% <ø> (ø)
Libplanet/Blocks/InvalidBlockIndexException.cs 100.00% <ø> (ø)
...planet/Blocks/InvalidBlockPreviousHashException.cs 100.00% <ø> (ø)
Libplanet/Blocks/InvalidBlockTimestampException.cs 100.00% <ø> (ø)
Libplanet/Blocks/InvalidGenesisBlockException.cs 71.42% <ø> (ø)
... and 113 more

moreal
moreal previously approved these changes Sep 27, 2020
Copy link
Contributor

@moreal moreal left a comment

Choose a reason for hiding this comment

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

LGTM 😎

longfin
longfin previously approved these changes Sep 27, 2020
CHANGES.md Outdated Show resolved Hide resolved
@dahlia
Copy link
Contributor Author

dahlia commented Sep 28, 2020

@planetarium/libplanet Added a missing reference to the PR number. Please review this again!

@dahlia dahlia merged commit a80daf6 into planetarium:main Sep 28, 2020
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.

6 participants