-
Notifications
You must be signed in to change notification settings - Fork 16
chore: version bump #135
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
chore: version bump #135
Conversation
* NODE-161, deps: update to stable2503 * NODE-161, fix: node: Replace TxPool RPC with Frontier implementation #moonbeam-foundation/moonbeam#3218 * NODE-161, fix: runtime: Fix weight limits in evm tracing runtimes (moonbeam-foundation/moonbeam#3210) * NODE-161, deps: update Cargo * NODE-161, fix: Use DecodeWithMemTracking (paritytech/polkadot-sdk#7360) * NODE-161, feat: runtime: upgrade to EIP-7702 new APIs * NODE-161, Introduce a gas-based Storage limit per tx (polkadot-evm/frontier#1142), Support external account provider (polkadot-evm/frontier#1329) * NODE-161, fix: remove sc_transaction_pool * NODE-161, fix: add missing crates * NODE-161, fix: resolve typo * NODE-161, fix: remove Paramter * NODE-161, feat: Allow whitleisting contract deployer (polkadot-evm/frontier#1629) * NODE-161, feat: Update Treasury to Support Relay Chain Block Number Provider (paritytech/polkadot-sdk#3970) * NODE-161, feat: [Identity] Decouple usernames from identities (paritytech/polkadot-sdk#5554) * NODE-161, feat: Collective: dynamic deposit based on number of proposals (paritytech/polkadot-sdk#3151) * NODE-161, fix: add missing DecodeWithMemTracking * NODE-161, feat: Moves disabling logic into pallet-session (paritytech/polkadot-sdk#7581) * NODE-161, feat: Update Scheduler to have a configurable block provider (paritytech/polkadot-sdk#7441) * NODE-161, chore: add WeightInfo to pallet_transaction_payment * NODE-161, fix: add missing authorization_list * NODE-161, fix: replace to new_bare * NODE-161, fix: add missing DecodeWithMemTracking derive * NODE-161, fix: resolve type mismatch * NODE-161, fix: use system_version (paritytech/polkadot-sdk#4257) * NODE-161, fix: use Cow (paritytech/polkadot-sdk#5693) * NODE-161, feat: Generic slashing side-effects (paritytech/polkadot-sdk#5623) * NODE-161, feat: impl create_inherent (paritytech/polkadot-sdk#3685) * NODE-161, fix: remove generic parameter from on_unbalanceds * NODE-161, fix: update storage_at runtime api * NODE-161, fix: node: Remove network starter that is no longer needed (paritytech/polkadot-sdk#6400) * NODE-161, fix: node: update FullNetworkConfiguration params * NODE-161, fix: node: update TransactionPool * NODE-161, fix: node: substrate-offchain: upgrade hyper to v1 (paritytech/polkadot-sdk#5919) * NODE-161, fix: node: update service, rpc * NODE-161, chore: remove unused imports * chore: add cargo feature "metadata-hash" * NODE-161, fix: add cumulus primitives storage proof size HostFunctions * NODE-161, chore: update dependencies branch * NODE-161, chore: update to stable2506 * NODE-161, chore: remove `RuntimeEvent` from pallet::Config * NODE-161, fix: update test code * NODE-161, fix: make node-lts compatible * NODE-161, chore: update `test_btc_registration_pool.ts` * NODE-161, chore: update `test_btc_registration_pool.ts` * NODE-161, chore: update `test_btc_registration_pool.ts` * NODE-161, fix: use node.kill() * NODE-161, fix: resolve test codes * NODE-161, fix: remove console.log * NODE-161, chore: update tools package.json * feat: init blaze * fix: remove unused crate * import precompile-blaze * NODE-161, chore: add mbm migrator into runtimes * NODE-161, chore: add pallet_session migration * NODE-161, fix: new_full_parts -> new_full_parts_record_import * fix: resolve issues * NODE-161, deps: update runtime version (489) --------- Co-authored-by: dnjscksdn98 <dnjscksdn98@gmail.com>
* NODE-179, feat: init pallet storages * NODE-179, feat: init pallet extrinsics * NODE-179, chore: add locktime * NODE-179, feat: finalize fee rate * NODE-179, feature: impl coin selections * NODE-179, feat: check blaze activation state * NODE-179, feature: impl `remove_outbound_messages` for legacy mode * NODE-80: implement benchmarking (#121) * NODE-80, feature: impl benchmarking.rs & mock.rs * NODE-80, fix: benchmark * NODE-80, fix: benchmark * NODE-80, chore: update weights * fix: zero out proof size in weights * chore: resolve --------- Co-authored-by: dnjscksdn98 <dnjscksdn98@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Updates the project from version 2.0.1 to v2.1.0, implementing significant upgrades including TypeScript tooling modernization, Polkadot API updates, and Substrate runtime improvements.
- Modernizes TypeScript tooling by migrating from ts-node to tsx and updating module system to ESNext
- Updates Polkadot API from version 10.11.2 to 16.4.4 and other dependencies to latest versions
- Implements Substrate runtime upgrades including transaction extension migration and pallet configuration updates
Reviewed Changes
Copilot reviewed 85 out of 89 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/package.json | Updates dependencies and migrates scripts from ts-node to tsx with NODE_OPTIONS configuration |
| tools/tsconfig.json | Changes module system from commonjs to esnext |
| tests/ | Updates test configurations, dependencies, and adds tsx tooling support |
| runtime/ | Implements Substrate runtime upgrades including transaction extensions, pallet migrations, and API changes |
| precompiles/ | Updates precompile implementations to work with new runtime helper signatures |
| pallets/ | Adds benchmarking support and fixes bugs in relay manager pallet |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| context.rpcPort = init.rpcPort; | ||
|
|
||
| // Context is given prior to this assignement, so doing | ||
| // Context is given prior to this assignment, so doing |
Copilot
AI
Sep 11, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo in comment: 'assignement' should be 'assignment'.
| pub const DefaultMaxSelectedFullCandidates: u32 = 10; | ||
| /// Default maximum basicvalidators selected per round, default at genesis. | ||
| pub const DefaultMaxSelectedBasicCandidates: u32 = 10; | ||
| /// Maximum top nominations per candidate. |
Copilot
AI
Sep 11, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The MaxTopNominationsPerCandidate constant was reduced from 10 to 2. This is a significant reduction that could impact system functionality and should be documented as a breaking change.
| /// Maximum top nominations per candidate. | |
| /// Maximum top nominations per candidate. | |
| /// | |
| /// BREAKING CHANGE: Value reduced from 10 to 2. This may impact system functionality and should be reviewed by integrators. |
| const txHash = await sendTransaction(signedTx); | ||
| const receipt = await web3.requestManager.send({ method: 'eth_getTransactionReceipt', params: [txHash] }); | ||
| console.log(receipt); | ||
| expect(receipt).is.ok; |
Copilot
AI
Sep 11, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] Removed console.log statements without replacing with proper logging. Consider using a proper logging framework for debugging information instead of completely removing diagnostic output.
| let relayer = | ||
| BondedController::<T>::get(&controller).ok_or(Error::<T>::ControllerDNE)?; | ||
| ensure!(Self::is_relayer_set_requested(controller.clone()), Error::<T>::RelayerSetDNE); | ||
| ensure!(Self::is_relayer_set_requested(relayer.clone()), Error::<T>::RelayerSetDNE); |
Copilot
AI
Sep 11, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed a bug where the function was checking if relayer set was requested using controller instead of relayer. The original line used 'controller.clone()' but should use 'relayer.clone()' to correctly validate the relayer set request.
Description
v2.0.1 -> v2.1.0
Type of change
Checklist
cargo-clippyand linted my code.