Format and Sort features in Cargo.toml files (#14803)

* CI: Add feature sorting check

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Sort all features

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add some mistakes

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Revert "Add some mistakes"

This reverts commit b2b1099f979f6decb22d09b46689c1554bb72e81.

* CI job naming

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add oneliner formatting

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Explain tool

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Use latest version

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Better erorr message

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Format after master merge

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Use --check option

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Messed up the merge commit...

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

substrate/frame/support/Cargo.toml

@@ -54,50 +54,53 @@ frame-system = { version = "4.0.0-dev", path = "../system" }
 array-bytes = "6.1"
 
 [features]
-default = ["std"]
+default = [ "std" ]
 std = [
-	"sp-core/std",
+	"codec/std",
+	"environmental/std",
+	"frame-metadata/std",
+	"frame-support-procedural/std",
+	"frame-system/std",
 	"k256/std",
+	"log/std",
+	"scale-info/std",
 	"serde/std",
 	"sp-api/std",
-	"sp-io/std",
-	"codec/std",
-	"scale-info/std",
-	"sp-std/std",
-	"sp-runtime/std",
-	"sp-tracing/std",
 	"sp-arithmetic/std",
-	"frame-metadata/std",
+	"sp-core/std",
+	"sp-debug-derive/std",
+	"sp-genesis-builder/std",
 	"sp-inherents/std",
+	"sp-io/std",
+	"sp-metadata-ir/std",
+	"sp-runtime/std",
 	"sp-staking/std",
 	"sp-state-machine/std",
+	"sp-std/std",
+	"sp-tracing/std",
 	"sp-weights/std",
-	"frame-support-procedural/std",
-	"log/std",
-	"environmental/std",
-	"sp-genesis-builder/std",
-	"frame-system/std",
-	"sp-debug-derive/std",
-	"sp-metadata-ir/std"
 ]
 runtime-benchmarks = [
 	"frame-system/runtime-benchmarks",
 	"sp-runtime/runtime-benchmarks",
-	"sp-staking/runtime-benchmarks"
+	"sp-staking/runtime-benchmarks",
 ]
 try-runtime = [
-	"sp-debug-derive/force-debug",
 	"frame-system/try-runtime",
-	"sp-runtime/try-runtime"
+	"sp-debug-derive/force-debug",
+	"sp-runtime/try-runtime",
 ]
 experimental = []
 # By default some types have documentation, `no-metadata-docs` allows to reduce the documentation
 # in the metadata.
-no-metadata-docs = ["frame-support-procedural/no-metadata-docs", "sp-api/no-metadata-docs"]
+no-metadata-docs = [
+	"frame-support-procedural/no-metadata-docs",
+	"sp-api/no-metadata-docs",
+]
 # By default some types have documentation, `full-metadata-docs` allows to add documentation to
 # more types in the metadata.
-full-metadata-docs = ["scale-info/docs"]
+full-metadata-docs = [ "scale-info/docs" ]
 # Generate impl-trait for tuples with the given number of tuples. Will be needed as the number of
 # pallets in a runtime grows. Does increase the compile time!
-tuples-96 = ["frame-support-procedural/tuples-96"]
-tuples-128 = ["frame-support-procedural/tuples-128"]
+tuples-96 = [ "frame-support-procedural/tuples-96" ]
+tuples-128 = [ "frame-support-procedural/tuples-128" ]

substrate/frame/support/procedural/Cargo.toml

@@ -28,7 +28,7 @@ macro_magic = { version = "0.4.2", features = ["proc_support"] }
 expander = "2.0.0"
 
 [features]
-default = ["std"]
+default = [ "std" ]
 std = []
 no-metadata-docs = []
 # Generate impl-trait for tuples with the given number of tuples. Will be needed as the number of

substrate/frame/support/test/Cargo.toml

@@ -37,33 +37,33 @@ frame-executive = { version = "4.0.0-dev", default-features = false, path = "../
 test-pallet = { package = "frame-support-test-pallet", default-features = false, path = "pallet" }
 
 [features]
-default = ["std"]
+default = [ "std" ]
 std = [
 	"codec/std",
 	"frame-benchmarking/std",
 	"frame-executive/std",
+	"frame-metadata/std",
 	"frame-support/std",
 	"frame-system/std",
 	"scale-info/std",
 	"serde/std",
 	"sp-api/std",
 	"sp-arithmetic/std",
-	"frame-metadata/std",
 	"sp-core/std",
 	"sp-io/std",
+	"sp-metadata-ir/std",
 	"sp-runtime/std",
+	"sp-state-machine/std",
 	"sp-std/std",
 	"sp-version/std",
 	"test-pallet/std",
-	"sp-state-machine/std",
-	"sp-metadata-ir/std"
 ]
-experimental = ["frame-support/experimental"]
+experimental = [ "frame-support/experimental" ]
 try-runtime = [
+	"frame-executive/try-runtime",
 	"frame-support/try-runtime",
 	"frame-system/try-runtime",
-	"frame-executive/try-runtime",
-	"sp-runtime/try-runtime"
+	"sp-runtime/try-runtime",
 ]
 # WARNING:
 # Only CI runs with this feature enabled. This feature is for testing stuff related to the FRAME macros
@@ -72,4 +72,4 @@ frame-feature-testing = []
 frame-feature-testing-2 = []
 # Disable ui tests
 disable-ui-tests = []
-no-metadata-docs = ["frame-support/no-metadata-docs"]
+no-metadata-docs = [ "frame-support/no-metadata-docs" ]

substrate/frame/support/test/compile_pass/Cargo.toml

@@ -21,11 +21,11 @@ sp-runtime = { version = "24.0.0", default-features = false, path = "../../../..
 sp-version = { version = "22.0.0", default-features = false, path = "../../../../primitives/version" }
 
 [features]
-default = ["std"]
+default = [ "std" ]
 std = [
 	"codec/std",
-	"renamed-frame-support/std",
 	"frame-system/std",
+	"renamed-frame-support/std",
 	"scale-info/std",
 	"sp-core/std",
 	"sp-runtime/std",

substrate/frame/support/test/pallet/Cargo.toml

@@ -20,12 +20,12 @@ frame-system = { version = "4.0.0-dev", default-features = false, path = "../../
 sp-runtime = { version = "24.0.0", default-features = false, path = "../../../../primitives/runtime" }
 
 [features]
-default = ["std"]
+default = [ "std" ]
 std = [
 	"codec/std",
 	"frame-support/std",
 	"frame-system/std",
 	"scale-info/std",
 	"serde/std",
-	"sp-runtime/std"
+	"sp-runtime/std",
 ]