Skip to content

Fix lint-staged script#3887

Merged
holgerd77 merged 1 commit into
masterfrom
fix-lintstaged
Feb 26, 2025
Merged

Fix lint-staged script#3887
holgerd77 merged 1 commit into
masterfrom
fix-lintstaged

Conversation

@acolytec3

Copy link
Copy Markdown
Contributor

This fixes an error condition in the lint-staged precommit git hook so that biome doesn't throw an error if it doesn't find any files to check.

@codecov

codecov Bot commented Feb 26, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.87%. Comparing base (a3c4c98) to head (a6e0d1a).
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

Flag Coverage Δ
block 76.87% <ø> (ø)
blockchain 85.69% <ø> (ø)
client 66.27% <ø> (ø)
common 90.73% <ø> (ø)
devp2p 76.27% <ø> (+0.06%) ⬆️
ethash 81.04% <ø> (ø)
evm 71.04% <ø> (ø)
genesis 99.84% <ø> (ø)
mpt 59.72% <ø> (ø)
rlp 69.70% <ø> (ø)
statemanager 70.47% <ø> (ø)
tx 80.96% <ø> (ø)
util 85.54% <ø> (ø)
vm 57.81% <ø> (ø)
wallet 83.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@holgerd77 holgerd77 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@holgerd77 holgerd77 merged commit 795bb13 into master Feb 26, 2025
@holgerd77 holgerd77 deleted the fix-lintstaged branch February 26, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants