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

Ensure cumulus/bridges is ignored by formatter and run it #1369

Merged
merged 1 commit into from
Sep 3, 2023

Conversation

gavofyork
Copy link
Member

Just a minor fix to .rustfmt.toml which had the incorrect path for bridges together with the few changes resulting from running it.

@gavofyork gavofyork added A1-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). T0-node This PR/Issue is related to the topic “node”. labels Sep 3, 2023
@gavofyork gavofyork changed the title Ensure cumulus/bridges is ignored by formatter and run it Ensure cumulus/bridges is ignored by formatter and run it Sep 3, 2023
@@ -19,9 +19,9 @@ trailing_comma = "Vertical"
trailing_semicolon = false
use_field_init_shorthand = true
ignore = [
"bridges",
"cumulus/bridges",
Copy link
Member

@ggwpez ggwpez Sep 3, 2023

Choose a reason for hiding this comment

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

@acatangiu this should be remove, or?
The current formatting files was merged as part of the monorepo import.

@gavofyork gavofyork merged commit bc6fca3 into master Sep 3, 2023
112 checks passed
@gavofyork gavofyork deleted the gav-fix-rustfmt-cfg branch September 3, 2023 13:11
ordian added a commit that referenced this pull request Sep 7, 2023
* master: (25 commits)
  Markdown linter (#1309)
  Update `fmt` file and some authors (#1379)
  Bump the known_good_semver group with 1 update (#1375)
  Bump proc-macro-warning from 0.4.1 to 0.4.2 (#1376)
  feat: add futures api to `TransactionPool` (#1348)
  Ensure cumulus/bridges is ignored by formatter and run it (#1369)
  substrate: chain-spec paths corrected in zombienet tests (#1362)
  contracts: Update to wasmi 0.31 (#1350)
  [improve docs]: Template pallet (#1280)
  [xcm-emulator] Unignore cumulus integration tests (#1247)
  Fix wrong ref counting (#1358)
  Use cached session index to obtain executor params (#1190)
  fix typos (#1339)
  Use bandersnatch-vrfs with locked dependencies ref (#1342)
  Bump bs58 from 0.4.0 to 0.5.0 (#1293)
  Contracts: `seal0::balance` should return the free balance (#1254)
  Logs: add extra debug log for negative rep changes (#1205)
  Added short-benchmarks for cumulus (#1183)
  [xcm-emulator] Improve hygiene and clean up (#1301)
  Bump the known_good_semver group with 1 update (#1347)
  ...
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A1-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants