ci(toml): check toml fmt (#2504)

## Which issue does this PR close?

<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases.
You can link an issue to this PR using the GitHub syntax. For example
`Closes #123` indicates that this PR will close issue #123.
-->

- Closes #2503

## What changes are included in this PR?

<!--
Provide a summary of the modifications in this PR. List the main changes
such as new features, bug fixes, refactoring, or any other updates.
-->

Taplo fmt check is added in `check-fmt` target in `Makefile`.

Format all TOML files.

## Are these changes tested?

<!--
Specify what test covers (unit test, integration test, etc.).

If tests are not included in your PR, please explain why (for example,
are they covered by existing tests)?
-->

Yes.

Author: TwinklerG

.typos.toml

@@ -19,8 +19,8 @@
 extend-ignore-identifiers-re = ["^bimap$"]
 
 [default.extend-words]
-ags = "ags"
 AGS = "AGS"
+ags = "ags"
 
 [files]
 extend-exclude = ["**/testdata", "CHANGELOG.md"]

Cargo.toml

@@ -56,8 +56,14 @@ arrow-string = "58"
 as-any = "0.3.2"
 async-trait = "0.1.89"
 aws-config = "1.8.7"
-aws-sdk-glue = { version = "1.85", default-features = false, features = ["default-https-client", "rt-tokio"] }
-aws-sdk-s3tables = { version = "1.28", default-features = false, features = ["default-https-client", "rt-tokio"] }
+aws-sdk-glue = { version = "1.85", default-features = false, features = [
+  "default-https-client",
+  "rt-tokio",
+] }
+aws-sdk-s3tables = { version = "1.28", default-features = false, features = [
+  "default-https-client",
+  "rt-tokio",
+] }
 backon = "1.5.1"
 base64 = "0.22.1"
 bimap = "0.6"
@@ -74,6 +80,10 @@ dirs = "6"
 enum-ordinalize = "4.3.0"
 env_logger = "0.11.8"
 expect-test = "1"
+fastnum = { version = "0.7", default-features = false, features = [
+  "std",
+  "serde",
+] }
 faststr = "0.2.31"
 flate2 = "1.1.5"
 fnv = "1.0.7"
@@ -113,7 +123,6 @@ regex = "1.11.3"
 reqwest = { version = "0.12.12", default-features = false, features = ["json"] }
 roaring = { version = "0.11" }
 rstest = "0.26"
-fastnum = { version = "0.7", default-features = false, features = ["std", "serde"] }
 serde = { version = "1.0.219", features = ["rc"] }
 serde_bytes = "0.11.17"
 serde_derive = "1.0.219"
@@ -126,7 +135,10 @@ stacker = "0.1.20"
 strum = "0.27.2"
 tempfile = "3.18"
 thrift = "0.17.0"
-tokio = { version = "1.47", default-features = false, features = ["sync", "rt-multi-thread"] }
+tokio = { version = "1.47", default-features = false, features = [
+  "sync",
+  "rt-multi-thread",
+] }
 toml = "0.8"
 tracing = "0.1.41"
 tracing-subscriber = "0.3.20"

Makefile

@@ -21,10 +21,10 @@ build:
 	cargo build --all-targets --all-features --workspace
 
 check-fmt:
-	cargo  fmt --all -- --check
+	cargo fmt --all -- --check
 
 check-clippy:
-	cargo  clippy --all-targets --all-features --workspace -- -D warnings
+	cargo clippy --all-targets --all-features --workspace -- -D warnings
 
 install-cargo-machete:
 	cargo install cargo-machete@0.7.0
@@ -45,7 +45,7 @@ fix-toml: install-taplo-cli
 	taplo fmt
 
 check-toml: install-taplo-cli
-	taplo check
+	taplo fmt --check
 
 NIGHTLY_VERSION := $(shell awk -F'"' '/^channel/ {print $$2}' rust-toolchain.toml)
 MSRV_VERSION    := $(shell awk -F'"' '/^rust-version/ {print $$2}' Cargo.toml)

bindings/python/Cargo.toml

@@ -32,11 +32,13 @@ crate-type = ["cdylib"]
 
 [dependencies]
 arrow = { version = "58", features = ["pyarrow", "chrono-tz"] }
+datafusion-ffi = "53.0.0"
 iceberg = { path = "../../crates/iceberg" }
-iceberg-storage-opendal = { path = "../../crates/storage/opendal", features = ["opendal-all"] }
-pyo3 = { version = "0.28", features = ["extension-module", "abi3-py310"] }
 iceberg-datafusion = { path = "../../crates/integrations/datafusion" }
-datafusion-ffi = "53.0.0"
+iceberg-storage-opendal = { path = "../../crates/storage/opendal", features = [
+  "opendal-all",
+] }
+pyo3 = { version = "0.28", features = ["extension-module", "abi3-py310"] }
 tokio = { version = "1.46.1", default-features = false }
 
 [profile.release]

bindings/python/pyproject.toml

@@ -33,20 +33,26 @@ classifiers = [
   "Programming Language :: Python :: 3.13",
   "Programming Language :: Rust",
 ]
+dynamic = ["version"]
+license = { file = "LICENSE" }
 name = "pyiceberg-core"
 readme = "project-description.md"
 requires-python = ">=3.10,<4"
-dynamic = ["version"]
-license = { file = "LICENSE" }
 
 [tool.maturin]
 features = ["pyo3/extension-module"]
-module-name = "pyiceberg_core.pyiceberg_core_rust"
-python-source = "python"
 include = [
-    { path = "LICENSE", format = ["sdist", "wheel"] },
-    { path = "NOTICE", format = ["sdist", "wheel"] }
+  { path = "LICENSE", format = [
+    "sdist",
+    "wheel",
+  ] },
+  { path = "NOTICE", format = [
+    "sdist",
+    "wheel",
+  ] },
 ]
+module-name = "pyiceberg_core.pyiceberg_core_rust"
+python-source = "python"
 
 [tool.ruff.lint]
 ignore = ["F403", "F405"]
@@ -60,15 +66,15 @@ filterwarnings = [
 
 [dependency-groups]
 dev = [
-    "maturin>=1.0,<2.0",
-    "pytest>=8.3.2",
-    "datafusion==52.*",
-    # Set a PyIceberg lower bound; otherwise the resolver may select the
-    # unrelated 0.0.2 stub.
-    # Keep pyarrow direct because current pyiceberg[pyarrow] releases depend on
-    # pyiceberg-core, which conflicts with this local pyiceberg-core project.
-    "pyiceberg[sql-sqlite]>=0.11",
-    "pyarrow>=17",
-    "fastavro>=1.11.1",
-    "huggingface_hub>=0.20",
+  "maturin>=1.0,<2.0",
+  "pytest>=8.3.2",
+  "datafusion==52.*",
+  # Set a PyIceberg lower bound; otherwise the resolver may select the
+  # unrelated 0.0.2 stub.
+  # Keep pyarrow direct because current pyiceberg[pyarrow] releases depend on
+  # pyiceberg-core, which conflicts with this local pyiceberg-core project.
+  "pyiceberg[sql-sqlite]>=0.11",
+  "pyarrow>=17",
+  "fastavro>=1.11.1",
+  "huggingface_hub>=0.20",
 ]

crates/catalog/loader/Cargo.toml

@@ -29,18 +29,18 @@ license = { workspace = true }
 repository = { workspace = true }
 
 [dependencies]
+async-trait = { workspace = true }
 iceberg = { workspace = true }
-iceberg-catalog-rest = { workspace = true }
 iceberg-catalog-glue = { workspace = true }
-iceberg-catalog-s3tables = { workspace = true }
 iceberg-catalog-hms = { workspace = true }
+iceberg-catalog-rest = { workspace = true }
+iceberg-catalog-s3tables = { workspace = true }
 iceberg-catalog-sql = { workspace = true }
 tokio = { workspace = true }
-async-trait = { workspace = true }
 
 [dev-dependencies]
-iceberg_test_utils = { path = "../../test_utils", features = ["tests"] }
 iceberg-storage-opendal = { workspace = true }
+iceberg_test_utils = { path = "../../test_utils", features = ["tests"] }
 reqwest = { workspace = true }
 rstest = { workspace = true }
 sqlx = { workspace = true, features = ["runtime-tokio", "sqlite", "migrate"] }

crates/catalog/s3tables/Cargo.toml

@@ -19,9 +19,9 @@
 edition = { workspace = true }
 homepage = { workspace = true }
 name = "iceberg-catalog-s3tables"
+readme = "README.md"
 rust-version = { workspace = true }
 version = { workspace = true }
-readme = "README.md"
 
 categories = ["database"]
 description = "Apache Iceberg Rust S3Tables Catalog"
@@ -37,7 +37,6 @@ aws-sdk-s3tables = { workspace = true }
 iceberg = { workspace = true }
 iceberg-storage-opendal = { workspace = true, features = ["opendal-s3"] }
 
-
 [dev-dependencies]
 arrow-array = { workspace = true }
 arrow-schema = { workspace = true }

crates/iceberg/Cargo.toml

@@ -31,7 +31,6 @@ repository = { workspace = true }
 [features]
 default = []
 
-
 [dependencies]
 aes-gcm = { workspace = true }
 anyhow = { workspace = true }
@@ -54,6 +53,7 @@ bytes = { workspace = true }
 chrono = { workspace = true }
 derive_builder = { workspace = true }
 expect-test = { workspace = true }
+fastnum = { workspace = true }
 flate2 = { workspace = true }
 fnv = { workspace = true }
 futures = { workspace = true }
@@ -66,7 +66,6 @@ parquet = { workspace = true, features = ["async", "encryption"] }
 rand = { workspace = true }
 reqwest = { workspace = true }
 roaring = { workspace = true }
-fastnum = { workspace = true }
 serde = { workspace = true }
 serde_bytes = { workspace = true }
 serde_derive = { workspace = true }
@@ -85,13 +84,13 @@ zstd = { workspace = true }
 [dev-dependencies]
 expect-test = { workspace = true }
 iceberg_test_utils = { path = "../test_utils", features = ["tests"] }
+minijinja = { workspace = true }
 mockall = { workspace = true }
 pretty_assertions = { workspace = true }
 rand = { workspace = true }
 regex = { workspace = true }
-tempfile = { workspace = true }
-minijinja = { workspace = true }
 serde_arrow = { version = "0.14", features = ["arrow-58"] }
+tempfile = { workspace = true }
 
 [package.metadata.cargo-machete]
 # These dependencies are added to ensure minimal dependency version

crates/integration_tests/Cargo.toml

@@ -32,7 +32,7 @@ iceberg = { workspace = true }
 iceberg-catalog-rest = { workspace = true }
 iceberg-storage-opendal = { workspace = true, features = ["opendal-s3"] }
 iceberg_test_utils = { path = "../test_utils", features = ["tests"] }
+ordered-float = "2.10.1"
 parquet = { workspace = true }
 tokio = { workspace = true }
 uuid = { workspace = true }
-ordered-float = "2.10.1"

crates/sqllogictest/Cargo.toml

@@ -35,11 +35,11 @@ iceberg = { workspace = true }
 iceberg-datafusion = { workspace = true }
 indicatif = { workspace = true }
 log = { workspace = true }
+serde = { workspace = true }
 sqllogictest = { workspace = true }
+tokio = { workspace = true }
 toml = { workspace = true }
-serde = { workspace = true }
 tracing = { workspace = true }
-tokio = { workspace = true }
 
 [dev-dependencies]
 libtest-mimic = { workspace = true }