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

Improve batch swap transfer #707

Merged
merged 6 commits into from
Sep 13, 2023
Merged

Improve batch swap transfer #707

merged 6 commits into from
Sep 13, 2023

Conversation

vovac12
Copy link
Contributor

@vovac12 vovac12 commented Sep 6, 2023

No description provided.

@vovac12 vovac12 requested review from wer1st and pakrentos September 6, 2023 09:38
[toolchain]
channel = "nightly-2023-03-21"
targets = [ "wasm32-unknown-unknown" ]
profile = "default"
Copy link
Contributor

Choose a reason for hiding this comment

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

add empty line please

@@ -0,0 +1,4 @@
[toolchain]
Copy link
Contributor

Choose a reason for hiding this comment

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

As I remember, there was a reason not to commit toolchain file. I could be wrong.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually it doesn't build without it. And I think we can use it to ensure that correct Rust version is used by everyone. If we will have any problems with this solution we can remove this file

@@ -0,0 +1,4 @@
[toolchain]
channel = "nightly-2023-03-21"
Copy link
Contributor

Choose a reason for hiding this comment

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

When we decide to change toolchain, we must update this line and this line.
Probably we want to do it now for #709

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have the same version of Rust in docker and we can't update it before Substrate update because of problems with wasm

rust-toolchain.toml Outdated Show resolved Hide resolved
@vovac12 vovac12 merged commit 3ef5d04 into develop Sep 13, 2023
@vovac12 vovac12 deleted the swapless-batch-swap-transfer branch September 13, 2023 09:25
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.

3 participants