Skip to content

Refactor: modernize format macros to use inlined arguments#32

Merged
Sid-352 merged 1 commit into
Sid-352:mainfrom
vaishnavirrapolu-06:modernize-format-macros
May 21, 2026
Merged

Refactor: modernize format macros to use inlined arguments#32
Sid-352 merged 1 commit into
Sid-352:mainfrom
vaishnavirrapolu-06:modernize-format-macros

Conversation

@vaishnavirrapolu-06
Copy link
Copy Markdown
Contributor

Description

This PR addresses issue #30 by modernizing format! and info! macros across the workspace to use inlined arguments as per Rust 1.58+ standards.The improvement makes the code more readable while following best practices in Rust. There have been no changes to the logic.

Related Issue

Closes #30

Checklist

  • I have read the CONTRIBUTING.md guidelines.
  • My code passes cargo clippy -- -D warnings and cargo fmt.
  • I have included rationale-only comments for complex logic.
  • I have added or updated unit tests (if applicable).

AI-Generated Code Compliance

  • This PR contains code generated or assisted by an LLM.
  • I have manually reviewed, tested, and verified every line of this code. Any issues faced, created or caused by the code are my own fault.
  • I can fully explain the low-level mechanics of this implementation and guarantee it does not introduce memory leaks, race conditions, or undefined behavior.

@vaishnavirrapolu-06
Copy link
Copy Markdown
Contributor Author

Hi @Sid-352,
Thank you for your time in reviewing this. I have successfully modernized the format macros to use inlined arguments as requested in #30.

Summary of changes:

  • Refactored the target macros to implement inlined argument usage.
  • Verified the changes locally to ensure functionality and stability.
  • Confirmed that this implementation adheres to the project's current modernization standards.

I’ve really enjoyed getting familiar with the codebase through this task. Once this PR is reviewed, I am very motivated to contribute to a Level 2 task. I noticed #13 and #31 are currently being worked on; please let me know if there is another feature or architectural task you would recommend I take on next!

@Sid-352 Sid-352 merged commit eb8490f into Sid-352:main May 21, 2026
1 check passed
@Sid-352
Copy link
Copy Markdown
Owner

Sid-352 commented May 21, 2026

Merged, thanks

@Sid-352 Sid-352 added nsoc26 Label for Nexus Spring of Code contributors level1 Simple addition, bugfix or change labels May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level1 Simple addition, bugfix or change nsoc26 Label for Nexus Spring of Code contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NSoC] Modernize format! macros across workspace

2 participants