Thanks to visit codestin.com
Credit goes to github.com

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

fix: prettier #355

Merged
merged 2 commits into from
Aug 18, 2021
Merged

fix: prettier #355

merged 2 commits into from
Aug 18, 2021

Conversation

ghost
Copy link

@ghost ghost commented Aug 18, 2021

Description

This PR fixes pre-commit hook by removing --write flag in yarn prettier.

Expected Behavior: pre-commit hook should exit with 1 if not formatted.
Actual Behavior: pre-commit hook formats code if not formatted

Also, it ignores coverage.

Motivation and Context

How has this been tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • Add tests to cover changes as needed.
  • Update documentation as needed.

@ghost ghost requested review from AmritKumar and bb111189 as code owners August 18, 2021 07:00
@codecov-commenter
Copy link

codecov-commenter commented Aug 18, 2021

Codecov Report

Merging #355 (c3c7dde) into dev (c8bc398) will increase coverage by 0.12%.
The diff coverage is 100.00%.

❗ Current head c3c7dde differs from pull request most recent head a2c4e18. Consider uploading reports for the commit a2c4e18 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #355      +/-   ##
==========================================
+ Coverage   80.67%   80.79%   +0.12%     
==========================================
  Files          44       44              
  Lines        1790     1802      +12     
  Branches      298      303       +5     
==========================================
+ Hits         1444     1456      +12     
  Misses        345      345              
  Partials        1        1              
Impacted Files Coverage Δ
packages/zilliqa-js-blockchain/src/chain.ts 65.07% <100.00%> (+2.15%) ⬆️
packages/zilliqa-js-core/src/net.ts 98.61% <100.00%> (+0.01%) ⬆️

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 c8bc398...a2c4e18. Read the comment docs.

@bb111189 bb111189 merged commit db41afa into dev Aug 18, 2021
@bb111189 bb111189 deleted the fix/prettier branch August 18, 2021 15:05
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