Skip to content

Conversation

@pm47
Copy link
Member

@pm47 pm47 commented Jul 16, 2021

No description provided.

@pm47 pm47 changed the title Re-enable fatal warnings Update dependencies and enable fatal warnings Jul 16, 2021
t-bast
t-bast previously approved these changes Jul 16, 2021
Co-authored-by: Bastien Teinturier <[email protected]>
@pm47 pm47 requested a review from t-bast July 16, 2021 16:45
@codecov-commenter
Copy link

Codecov Report

Merging #1885 (d69b455) into master (79729c7) will increase coverage by 0.00%.
The diff coverage is 74.79%.

@@           Coverage Diff           @@
##           master    #1885   +/-   ##
=======================================
  Coverage   87.31%   87.31%           
=======================================
  Files         159      159           
  Lines       11989    11995    +6     
  Branches      501      459   -42     
=======================================
+ Hits        10468    10474    +6     
  Misses       1521     1521           
Impacted Files Coverage Δ
...n/scala/fr/acinq/eclair/balance/BalanceActor.scala 11.32% <0.00%> (ø)
...n/scala/fr/acinq/eclair/balance/CheckBalance.scala 57.47% <0.00%> (-1.36%) ⬇️
...lair/blockchain/watchdogs/BlockchainWatchdog.scala 57.69% <0.00%> (ø)
...-core/src/main/scala/fr/acinq/eclair/package.scala 75.55% <0.00%> (ø)
...scala/fr/acinq/eclair/payment/PaymentRequest.scala 92.22% <ø> (ø)
...scala/fr/acinq/eclair/payment/send/Autoprobe.scala 0.00% <0.00%> (ø)
...in/scala/fr/acinq/eclair/router/NetworkStats.scala 100.00% <ø> (ø)
...cala/fr/acinq/eclair/wire/protocol/TlvCodecs.scala 100.00% <ø> (ø)
...scala/fr/acinq/eclair/wire/protocol/TlvTypes.scala 100.00% <ø> (ø)
...nt/src/main/scala/fr/acinq/eclair/FrontSetup.scala 0.00% <0.00%> (ø)
... and 37 more

@pm47 pm47 merged commit c22596b into master Jul 16, 2021
@pm47 pm47 deleted the fatal-warnings branch July 16, 2021 17:02
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.

5 participants