Skip to content
This repository was archived by the owner on Jan 20, 2023. It is now read-only.

Update Kotlin and Improvement of initialization process.#14

Merged
k163377 merged 43 commits intoProjectMapK:masterfrom
k163377:update_kotlin
Dec 20, 2020
Merged

Update Kotlin and Improvement of initialization process.#14
k163377 merged 43 commits intoProjectMapK:masterfrom
k163377:update_kotlin

Conversation

@k163377
Copy link
Contributor

@k163377 k163377 commented Dec 20, 2020

ライブラリアップデート/導入

  • Kotlin 1.4.21
  • io.mockk:mockk:1.10.3-jdk8

バグ修正

  • 引数無しで呼び出せる関数を渡した場合に範囲外アクセスで落ちていた問題の修正

初期化処理の改善

  • インスタンスパラメータに対するバリデーションを追加し、不正なインスタンスを渡した場合にIllegalArgumentExceptionで落とすよう改善した

その他

  • Kotlinのアップデートに伴うベンチマーク更新[
    • 特に顕著なスコア変動は無かった
  • テストの拡充

@codecov-io
Copy link

Codecov Report

Merging #14 (5465444) into master (cefe218) will increase coverage by 6.87%.
The diff coverage is 92.50%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #14      +/-   ##
============================================
+ Coverage     81.90%   88.78%   +6.87%     
  Complexity       28       28              
============================================
  Files             8        8              
  Lines           199      214      +15     
  Branches         36       35       -1     
============================================
+ Hits            163      190      +27     
+ Misses           26       20       -6     
+ Partials         10        4       -6     
Impacted Files Coverage Δ Complexity Δ
...kotlin/com/mapk/fastkfunction/InternalFunctions.kt 83.33% <80.00%> (-16.67%) 0.00 <0.00> (ø)
...n/com/mapk/fastkfunction/SingleArgFastKFunction.kt 86.44% <88.23%> (+13.36%) 0.00 <0.00> (ø)
...ain/kotlin/com/mapk/fastkfunction/FastKFunction.kt 98.87% <100.00%> (+8.17%) 1.00 <0.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 cefe218...5465444. Read the comment docs.

@k163377 k163377 merged commit 12e6e29 into ProjectMapK:master Dec 20, 2020
@k163377 k163377 deleted the update_kotlin branch December 26, 2020 05:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants