Skip to content

Fix formatting - #177

Open
lforchini wants to merge 2 commits into
mainfrom
fix-formatting2
Open

lforchini wants to merge 2 commits into
mainfrom
fix-formatting2

Conversation

@lforchini

Copy link
Copy Markdown
Collaborator

Signed-off-by: Leonardo Forchini leonardo.forchini@nutanix.com


Stack created with GitHub Stacks CLI • Give Feedback 💬

Signed-off-by: Leonardo Forchini <leonardo.forchini@nutanix.com>

@tmakatos tmakatos left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this something make check caught? If so, shouldn't CI have failed and block merging this commit?

@lforchini

Copy link
Copy Markdown
Collaborator Author

All my other PRs were failing CI because of this formatting issue. CI failed on #168 but the PR was still mergeable.

@lforchini
lforchini removed this pull request from stack #178 November 6, 2026 10:23
@lforchini
lforchini added this pull request to stack #180 November 6, 2026 10:24
@lforchini

Copy link
Copy Markdown
Collaborator Author

The settings should've been updated now, we shouldn't be able to merge without all the checks passing.

It appears that the new Rust 1.100 has changed how comments are wrapped.
Fix the new formatting violations.

Signed-off-by: Leonardo Forchini <leonardo.forchini@nutanix.com>
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