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

Add version to tracing runtimes #995

Merged
merged 1 commit into from
Aug 7, 2023
Merged

Add version to tracing runtimes #995

merged 1 commit into from
Aug 7, 2023

Conversation

bLd75
Copy link
Contributor

@bLd75 bLd75 commented Aug 4, 2023

Pull Request Summary

Include runtime version in EVM tracing blobs

Check list

  • added or updated unit tests
  • updated Astar official documentation
  • added OnRuntimeUpgrade hook for precompile revert code registration
  • updated spec version
  • updated semver

@bLd75 bLd75 added the ci This PR/Issue is related to the topic "CI" label Aug 4, 2023
@github-actions
Copy link

github-actions bot commented Aug 4, 2023

Code Coverage

Package Line Rate Branch Rate Health
precompiles/assets-erc20/src 76% 0%
pallets/xvm/src 0% 0%
precompiles/sr25519/src 79% 0%
precompiles/substrate-ecdsa/src 78% 0%
precompiles/utils/src 70% 0%
chain-extensions/pallet-assets/src 0% 0%
pallets/block-reward/src 85% 0%
pallets/ethereum-checked/src 48% 0%
pallets/custom-signatures/src 51% 0%
chain-extensions/types/dapps-staking/src 0% 0%
chain-extensions/xvm/src 0% 0%
pallets/pallet-xcm/src 53% 0%
pallets/contracts-migration/src 0% 0%
chain-extensions/types/assets/src 0% 0%
pallets/dapps-staking/src/pallet 85% 0%
pallets/dapps-staking/src 81% 0%
primitives/src/xcm 67% 0%
pallets/xc-asset-config/src 54% 0%
precompiles/xcm/src 84% 0%
chain-extensions/types/xvm/src 0% 0%
chain-extensions/dapps-staking/src 0% 0%
pallets/collator-selection/src 69% 0%
precompiles/dapps-staking/src 93% 0%
precompiles/utils/macro/src 0% 0%
precompiles/xvm/src 67% 0%
primitives/src 69% 0%
Summary 55% (2271 / 4108) 0% (0 / 0)

Minimum allowed line rate is 50%

Copy link
Member

@Dinonard Dinonard left a comment

Choose a reason for hiding this comment

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

It would be helpful to describe in the summary what exactly was changed.
E.g. format-x to formatY

@Dinonard Dinonard merged commit 10d0326 into master Aug 7, 2023
13 checks passed
@Dinonard Dinonard deleted the feat/ci-tracing-update branch August 7, 2023 14:05
PierreOssun pushed a commit that referenced this pull request Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This PR/Issue is related to the topic "CI"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants