Skip to content

Conversation

nirosys
Copy link
Contributor

@nirosys nirosys commented Oct 9, 2025

Issue #, if available: n/a

Description of changes:
This PR is just the result of running rustfmt. With the changes made in #1006, the rustfmt check was re-enabled after some period of not being run in the workflow resulting in failures due to rustfmt diffs being detected.

In a few cases I added in #[rustfmt::skip] in order to maintain spacing in comments for multi-line byte slices that contained ion text equivalent comments. Otherwise, the changes are all rustfmt provided.

Before PR

% cargo fmt --check --message-format 'short'
/Users/glitch/Code/sandbox/ion-rust/benches/byod.rs
/Users/glitch/Code/sandbox/ion-rust/benches/encoding_primitives.rs
/Users/glitch/Code/sandbox/ion-rust/benches/write_many_structs.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/binary/raw/v1_1/binary_buffer.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/binary/raw/v1_1/e_expression.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/binary/raw/value.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/encoder/binary/v1_1/container_writers.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/encoder/binary/v1_1/fixed_uint.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/encoder/binary/v1_1/value_writer.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/encoder/writer.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/expanded/e_expression.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/expanded/macro_evaluator.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/expanded/template.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/reader.rs
/Users/glitch/Code/sandbox/ion-rust/src/lazy/struct.rs
/Users/glitch/Code/sandbox/ion-rust/src/serde/de.rs
/Users/glitch/Code/sandbox/ion-rust/src/serde/mod.rs
/Users/glitch/Code/sandbox/ion-rust/src/serde/ser.rs
/Users/glitch/Code/sandbox/ion-rust/src/types/decimal/mod.rs
/Users/glitch/Code/sandbox/ion-rust/src/types/mod.rs
/Users/glitch/Code/sandbox/ion-rust/tests/conformance_dsl/model.rs
/Users/glitch/Code/sandbox/ion-rust/tests/conformance_dsl/model.rs
/Users/glitch/Code/sandbox/ion-rust/tests/conformance_tests.rs
/Users/glitch/Code/sandbox/ion-rust/tests/ion_hash_tests.rs
% cargo fmt --check --message-format 'short' | wc -l
      24

After PR

% cargo fmt --check --message-format 'short'
%

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@nirosys nirosys marked this pull request as ready for review October 9, 2025 23:28
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.

1 participant