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 deps #1327

Merged
merged 1 commit into from
Aug 19, 2024
Merged

remove unused deps #1327

merged 1 commit into from
Aug 19, 2024

Conversation

ermalkaleci
Copy link
Contributor

Pull Request Summary

@ermalkaleci ermalkaleci added the other PR/issue is related to third-party, scripts or other things which don't have definite label. label Aug 18, 2024
Copy link

Code Coverage

Package Line Rate Branch Rate Health
pallets/price-aggregator/src 82% 0%
pallets/dapp-staking-migration/src 0% 0%
pallets/xc-asset-config/src 50% 0%
pallets/astar-xcm-benchmarks/src 86% 0%
pallets/dynamic-evm-base-fee/src 89% 0%
precompiles/unified-accounts/src 100% 0%
pallets/dapp-staking-v3/src/benchmarking 98% 0%
pallets/collator-selection/src 92% 0%
chain-extensions/types/unified-accounts/src 0% 0%
chain-extensions/unified-accounts/src 0% 0%
pallets/dapp-staking-v3/src 84% 0%
pallets/unified-accounts/src 86% 0%
precompiles/xcm/src 71% 0%
pallets/astar-xcm-benchmarks/src/fungible 100% 0%
primitives/src/xcm 65% 0%
pallets/dapp-staking-v3/src/test 0% 0%
pallets/astar-xcm-benchmarks/src/generic 100% 0%
precompiles/assets-erc20/src 78% 0%
pallets/collective-proxy/src 86% 0%
pallets/inflation/src 93% 0%
pallets/static-price-provider/src 85% 0%
precompiles/dapp-staking-v3/src/test 0% 0%
precompiles/sr25519/src 64% 0%
primitives/src 59% 0%
pallets/dapp-staking-v3/rpc/runtime-api/src 0% 0%
pallets/ethereum-checked/src 74% 0%
chain-extensions/pallet-assets/src 56% 0%
chain-extensions/types/assets/src 0% 0%
precompiles/dapp-staking-v3/src 90% 0%
precompiles/dispatch-lockdrop/src 86% 0%
precompiles/substrate-ecdsa/src 74% 0%
Summary 77% (3565 / 4627) 0% (0 / 0)

Minimum allowed line rate is 50%

Copy link
Contributor

@ipapandinas ipapandinas left a comment

Choose a reason for hiding this comment

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

LGTM

@ermalkaleci ermalkaleci merged commit 40c4b6c into master Aug 19, 2024
8 of 9 checks passed
@ermalkaleci ermalkaleci deleted the fix/deps-cleanup branch August 19, 2024 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
other PR/issue is related to third-party, scripts or other things which don't have definite label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants