Skip to content

Conversation

imbajin
Copy link
Member

@imbajin imbajin commented Jun 8, 2023

Purpose of the PR

we should keep dev list clean (currently still too much, next time change server issue) 📪

Main Changes

Verifying these changes

  • This change is a trivial rework / code cleanup without any test coverage.

(or)

  • This change is already covered by existing tests, such as (please describe tests).

(or)

  • This change added tests and can be verified as follows:

    (for example:)

    • Add UT.

Does this PR potentially affect the following parts?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other affects (typed here)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - NO Need
@codecov
Copy link

codecov bot commented Jun 8, 2023

Codecov Report

Merging #255 (6c46686) into master (e62ff3f) will decrease coverage by 0.02%.
The diff coverage is 88.88%.

@@ Coverage Diff @@ ## master #255 +/- ## ============================================ - Coverage 85.81% 85.79% -0.02%  + Complexity 3234 3233 -1  ============================================ Files 344 344 Lines 12108 12115 +7 Branches 1090 1092 +2 ============================================ + Hits 10390 10394 +4  Misses 1196 1196 - Partials 522 525 +3 
Impacted Files Coverage Δ
...rg/apache/hugegraph/computer/core/bsp/BspBase.java 97.72% <85.71%> (+5.41%) ⬆️
...ugegraph/computer/core/config/ComputerOptions.java 98.89% <100.00%> (+<0.01%) ⬆️

... and 1 file with indirect coverage changes

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

@imbajin imbajin merged commit 024d1aa into master Jun 9, 2023
@imbajin imbajin deleted the imbajin-patch-1 branch June 9, 2023 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants