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

Upgrade to Polkadot v1.1.0 #1403

Merged
merged 9 commits into from
Oct 10, 2023
Merged

Upgrade to Polkadot v1.1.0 #1403

merged 9 commits into from
Oct 10, 2023

Conversation

@jasl jasl requested a review from kvinwang October 8, 2023 08:35
@kvinwang kvinwang force-pushed the release-polkadot-v1.1.0 branch from c253492 to de6a04f Compare October 9, 2023 05:37
@kvinwang kvinwang force-pushed the release-polkadot-v1.1.0 branch from de6a04f to a15217f Compare October 9, 2023 08:49
@jasl jasl force-pushed the release-polkadot-v1.1.0 branch from 039c7c7 to 15ee1f7 Compare October 9, 2023 19:05
@jasl jasl changed the title Release polkadot v1.1.0 Upgrade to Polkadot v1.1.0 Oct 10, 2023
standalone/runtime/src/lib.rs Show resolved Hide resolved
telemetry.as_ref().map(|(_, telemetry)| telemetry.handle()),
executor,
)?;
let client = Arc::new(client);
Copy link
Collaborator

Choose a reason for hiding this comment

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

So, does the upstream switched to spaces from tabs for indentation?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It should use tabs, our editor config is also configured using tabs.
However, currently, most of our codes are using space, so I still format to use spaces here.

It's better to make a separate PR for reformat to use tabs

@jasl jasl merged commit 1d88376 into master Oct 10, 2023
8 checks passed
@jasl jasl deleted the release-polkadot-v1.1.0 branch October 10, 2023 08:23
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.

2 participants