Skip to content

ref: Reformat src/commands#2621

Merged
szokeasaurusrex merged 1 commit intomasterfrom
szokeasaurusrex/format-commands
Jul 16, 2025
Merged

ref: Reformat src/commands#2621
szokeasaurusrex merged 1 commit intomasterfrom
szokeasaurusrex/format-commands

Conversation

@szokeasaurusrex
Copy link
Copy Markdown
Member

@szokeasaurusrex szokeasaurusrex commented Jul 16, 2025

Previously, cargo fmt was unable to find these files, so CI could not enforce formatting. Fortunately, seems as though most contributors have been using rust-analyzer locally, so there are only a few formatter violations to fix here.

Prerequisite for #2622

Previously, `cargo fmt` was unable to find these files, so CI could not enforce formatting. Fortunately, seems as though most contributors have been using rust-analyzer locally, so there are only a few formatter violations to fix here.
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/format-commands branch 2 times, most recently from 67f0296 to 81e940a Compare July 16, 2025 14:14
@szokeasaurusrex szokeasaurusrex changed the title ref: Define command modules manually ref: Reformat src/commands.rs Jul 16, 2025
Copy link
Copy Markdown
Contributor

@loewenheim loewenheim left a comment

Choose a reason for hiding this comment

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

PR itself looks good, but the title is a bit off—no file named commands.rs exists in the repo.

@szokeasaurusrex szokeasaurusrex changed the title ref: Reformat src/commands.rs ref: Reformat src/commands Jul 16, 2025
@szokeasaurusrex
Copy link
Copy Markdown
Member Author

Yeah, i meant just src/commands. Thanks for catching

@szokeasaurusrex szokeasaurusrex merged commit ea33ed8 into master Jul 16, 2025
26 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/format-commands branch July 16, 2025 14:19
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.

2 participants