Skip to content
This repository has been archived by the owner on May 26, 2023. It is now read-only.

ESLint no-unreachable #320

Merged
merged 2 commits into from Jul 29, 2021
Merged

ESLint no-unreachable #320

merged 2 commits into from Jul 29, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jul 29, 2021

Description

This PR uses ESLint no-unreachable rule and removes the unreachable codes.

Review Suggestion

Status

Implementation

  • ready for review

@ghost ghost requested review from AmritKumar, bb111189, renlulu and teye as code owners July 29, 2021 08:11
@ghost ghost force-pushed the refactor/no-unreachable branch from 04cd8c4 to b98ee44 Compare July 29, 2021 08:13
@ghost ghost changed the title Refactor/no unreachable no-unreachable Jul 29, 2021
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2021

Codecov Report

Merging #320 (b98ee44) into dev (0f60d91) will increase coverage by 0.17%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #320      +/-   ##
==========================================
+ Coverage   79.02%   79.19%   +0.17%     
==========================================
  Files          45       45              
  Lines        1821     1817       -4     
  Branches      306      304       -2     
==========================================
  Hits         1439     1439              
+ Misses        381      377       -4     
  Partials        1        1              
Impacted Files Coverage Δ
packages/zilliqa-js-util/src/validation.ts 77.77% <ø> (+6.34%) ⬆️

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 0f60d91...b98ee44. Read the comment docs.

@ghost ghost changed the title no-unreachable ESLint no-unreachable Jul 29, 2021
@bb111189 bb111189 merged commit 8aee5aa into dev Jul 29, 2021
@ghost ghost deleted the refactor/no-unreachable branch August 17, 2021 11:27
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.

None yet

2 participants