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

Optimized latest Polkadot library #270

Draft
wants to merge 331 commits into
base: main
Choose a base branch
from
Draft

Optimized latest Polkadot library #270

wants to merge 331 commits into from

Conversation

ndkazu
Copy link
Contributor

@ndkazu ndkazu commented Aug 1, 2023

using polkadot-sdk/substrate master branch

apopiak and others added 30 commits November 10, 2020 15:56
The instruction on readme explicitly depends on nightly-2020-10-05
Update init.sh to reflect working nightly
* Update Local Development docs to use Makefile

* Update Makefile per @athei

Co-authored-by: Alexander Theißen <alex.theissen@me.com>

Co-authored-by: Alexander Theißen <alex.theissen@me.com>
* Add .editorconfig and format code style

* Enforce line length to be smaller than 100
* Add Rust setup steps

* Update to v2.0.1

* Cannot skip Wasm build

* Update docs

* Update build.rs

* new wasm builder, cargo update, and script tweaks

* minor update

Co-authored-by: Dan Shields <35669742+NukeManDan@users.noreply.github.com>
Co-authored-by: Dan Shields <nukemandan@protonmail.com>
Co-authored-by: Jimmy Chu <jimmychu0807@gmail.com>
* Patch Multiaddress Into Substrate Node Template

* Updated all version to 100

* Update runtime/src/lib.rs

* Update runtime/src/lib.rs

Co-authored-by: Jimmy Chu <jimmychu0807@gmail.com>
Add Fedora/RHEL/CentOS Installation details
Added button and text to point reader to it Playground.
Include Playground link in Node Template README
This commit removes the `AccountIndex` from the runtime/src/lib.rs as the `AccountIndex` type is typically used to configure the Indices pallet (https://substrate.dev/rustdocs/v3.0.0/pallet_indices/trait.Config.html) which is not used by the node template, so there is no point to include it.
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.