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

Merged Parachain repo #870

Merged
merged 64 commits into from
Nov 1, 2023
Merged

Merged Parachain repo #870

merged 64 commits into from
Nov 1, 2023

Conversation

zktony
Copy link
Contributor

@zktony zktony commented Sep 12, 2023

Describe your changes

Merged Parachain repo.

  • Fix try-runtime error.
  • Generate new weights.

@zktony zktony changed the base branch from mainnet-release to Develop September 12, 2023 10:17
@pavankanteti pavankanteti added the A0-PleaseReview Pull request needs code review. label Sep 20, 2023
@pavankanteti pavankanteti requested review from serhii-temchenko and removed request for 35359595 September 20, 2023 10:03
Copy link
Contributor

@serhii-temchenko serhii-temchenko left a comment

Choose a reason for hiding this comment

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

Plus one question - since bumped version of the code libraries - shouldn't weights be updated?

.docker/ci/release/Dockerfile Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
scripts/start_chain.sh Outdated Show resolved Hide resolved
scripts/start_chain.sh Outdated Show resolved Hide resolved
scripts/start_chain.sh Outdated Show resolved Hide resolved
scripts/start_chain.sh Outdated Show resolved Hide resolved
scripts/start_chain.sh Outdated Show resolved Hide resolved
serhii-temchenko and others added 2 commits October 27, 2023 00:20
# Conflicts:
#	runtimes/mainnet/Cargo.toml
#	runtimes/mainnet/src/impls.rs
serhii-temchenko and others added 23 commits October 27, 2023 01:34
# Conflicts:
#	runtimes/parachain/Cargo.toml
## Describe your changes
After substrate version update in PR #870 weights of pallets should be
updated.
In this PR newly generated weights are provided. 

## Issue ticket number and link
[#241](Polkadex-Substrate/issue-tracker#241)

## Checklist before requesting a review
- [X] I have performed a self-review of my code.
- [X] If it is a core feature, I have added thorough tests.
- [X] I removed all Clippy and Formatting Warnings. 
- [X] I added required Copyrights.

~~:warning: Note that parachain related pallets weights (e.g.
`thea-council` and `xcm-helper`) didn't updated because of error:~~

> ~~Error: Input("Did not find the benchmarking metadata. This could
mean that you either did not build the node correctly with the
`--features runtime-benchmarks` flag, or the chain spec that you are
using was not created by a node that was compiled with the flag: Other:
Exported method Benchmark_benchmark_metadata is not found")~~

~~Even though `--features runtime-benchmarks` flag provided on the build
process.
@zktony please check this problem.~~ Fixed.
@Gauthamastro Gauthamastro merged commit db5fd2c into Develop Nov 1, 2023
@Gauthamastro Gauthamastro deleted the zk-reorg branch November 1, 2023 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-PleaseReview Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants