Skip to content

Configure rustfmt hex_literal_case - #571

Open
nikobockerman wants to merge 1 commit into
mainfrom
configure-rustfmt-hex-literal-case
Open

Configure rustfmt hex_literal_case#571
nikobockerman wants to merge 1 commit into
mainfrom
configure-rustfmt-hex-literal-case

Conversation

@nikobockerman

Copy link
Copy Markdown
Owner

This configuration option is recently stabilized and defaults to "Preserve". Configure it to "Upper" in order to have unified formatting in the repo.

Copilot AI lite review requested due to automatic review settings June 16, 2026 04:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a Rust formatting configuration to enforce consistent casing for hexadecimal literals across the Rust solver codebase.

Changes:

  • Introduces hex_literal_case = "Upper" in the Rust solver’s rustfmt.toml to standardize hex literal formatting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

This configuration option is recently stabilized and defaults to
"Preserve". Configure it to "Upper" in order to have unified formatting
in the repo.
Copilot AI review requested due to automatic review settings August 22, 2026 14:40
@nikobockerman
nikobockerman force-pushed the configure-rustfmt-hex-literal-case branch from 5c6c719 to 9ae5aa5 Compare August 22, 2026 14:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change is a scoped rustfmt configuration update and is consistent with the Rust solver crate’s formatting workflow.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

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