Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused dependencies #95

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Remove unused dependencies #95

merged 1 commit into from
Jan 17, 2025

Conversation

pmikolajczyk41
Copy link
Member

some code gardening

@pmikolajczyk41 pmikolajczyk41 requested a review from kroist January 17, 2025 16:31
Copy link

Transaction NameMainCurrentDifference (%)
NewAccountNative20003232000263-0.00300%
DepositNative18278941827834-0.00328%
WithdrawNative18981861898078-0.00569%

Copy link

📊 Coverage Report

📈 Total Coverage Summary

Type Covered Total Coverage
📝 Lines 142 403 ❌ 35.23%
📄 Statements 143 406 ❌ 35.22%
⚡ Functions 38 105 ❌ 36.19%
🔀 Branches 14 66 ❌ 21.21%

Coverage Legend

  • ✅ 100% Coverage
  • 🟡 80-99% Coverage
  • 🟠 50-79% Coverage
  • ❌ 0-49% Coverage

📁 File Coverage

📋 Detailed Coverage Report
File Lines Statements Functions Branches
zkOS-monorepo/ts/shielder-sdk/src/actions/deposit.ts 100% 100% 100% 100%
zkOS-monorepo/ts/shielder-sdk/src/actions/newAccount.ts 100% 100% 100% 100%
zkOS-monorepo/ts/shielder-sdk/src/actions/utils.ts 100% 100% 100% 100%
zkOS-monorepo/ts/shielder-sdk/src/actions/withdraw.ts 100% 100% 100% 100%
zkOS-monorepo/ts/shielder-sdk/src/chain/contract.ts 4.76% 4.76% 7.14% 0%
zkOS-monorepo/ts/shielder-sdk/src/chain/relayer.ts 8.57% 8.57% 14.28% 0%
zkOS-monorepo/ts/shielder-sdk/src/client.ts 0% 0% 0% 0%
zkOS-monorepo/ts/shielder-sdk/src/constants.ts 100% 100% 100% 100%
zkOS-monorepo/ts/shielder-sdk/src/state/events.ts 0% 0% 0% 0%
zkOS-monorepo/ts/shielder-sdk/src/state/manager.ts 0% 0% 0% 0%
zkOS-monorepo/ts/shielder-sdk/src/state/storageSchema.ts 0% 0% 0% 0%
zkOS-monorepo/ts/shielder-sdk/src/state/sync.ts 0% 0% 0% 0%
zkOS-monorepo/ts/shielder-sdk/src/utils.ts 100% 100% 100% 100%

@pmikolajczyk41 pmikolajczyk41 merged commit f0c512d into main Jan 17, 2025
9 checks passed
@pmikolajczyk41 pmikolajczyk41 deleted the deps branch January 17, 2025 16:49
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.

2 participants