Skip to content

refactor: harmonize functions to format/functions.rs - #943

Merged
Its-Just-Nans merged 36 commits into
masterfrom
harmonize-functions
Aug 17, 2026
Merged

refactor: harmonize functions to format/functions.rs#943
Its-Just-Nans merged 36 commits into
masterfrom
harmonize-functions

Conversation

@Its-Just-Nans

Copy link
Copy Markdown
Member

@amazon-q-developer amazon-q-developer Bot 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.

The refactoring looks solid overall. The code has been reorganized to consolidate format-related functions into src/format/functions.rs, which improves code structure. The logic appears to be correctly preserved across the move, with no functional changes beyond better organization.

One minor issue found: a typo in a comment that should be corrected.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

Comment thread src/format/mod.rs Outdated
@Its-Just-Nans Its-Just-Nans mentioned this pull request Aug 17, 2026
2 tasks
@Its-Just-Nans
Its-Just-Nans added this pull request to the merge queue Aug 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 17, 2026
@Its-Just-Nans
Its-Just-Nans added this pull request to the merge queue Aug 17, 2026
@Its-Just-Nans

Copy link
Copy Markdown
Member Author

Seems like github is githubing (per https://www.githubstatus.com/)

Merged via the queue into master with commit 3f36785 Aug 17, 2026
134 checks passed
@Its-Just-Nans
Its-Just-Nans deleted the harmonize-functions branch August 17, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant