Addition of rustfmt run over generated test vectors.#112
Conversation
vivek-arte
force-pushed
the
formatting_updates_1
branch
from
9357318 to
9e6575a
Compare
oxarbitrage
left a comment
oxarbitrage
left a comment
There was a problem hiding this comment.
Verified that running rustfmt --edition 2021 test-vectors/rust/*.rs on the master branch files produces the output in this PR. Addition of the command to regenrate.sh looks good so rust test vectors are consistently formatted for direct use in other repositories. Approving, i think we should merge.
|
Need rebase now that we have #114 merged. |
…ying into required repositories
vivek-arte
force-pushed
the
formatting_updates_1
branch
from
9e6575a to
6988e59
Compare
|
@oxarbitrage I rebased, and the CI passes on the local PR (QED-it#41). |
|
This change is no longer needed — PR #122 (merged to master) already runs uv run $generator -t $gen_type | rustfmt >test-vectors/$gen_type/$generator.$extensionThis achieves the same effect of ensuring all generated Rust test vectors are formatted. 🤖 Comment written with Claude Code |
|
Closing per above comment. I will add |
As suggested by @vivek-arte in #112. This does not change the output with the current version of rustfmt but is good practice for forward compatibility. Co-Authored-By: Vivek Arte <46618816+vivek-arte@users.noreply.github.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
3 participants
This PR adds a step to run
rustfmtover the generated Rust test vectors.This allows the vectors to be copied directly to where they are used (in other repositories) with minimal post copy work.
This was suggested to be separated out of #108 in #108 (comment).