Skip to content

Reflow comments with make indent - #13

Merged
jserv merged 1 commit into
mainfrom
reflow
Oct 5, 2026
Merged

jserv merged 1 commit into
mainfrom
reflow

Conversation

@jserv

@jserv jserv commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Runs make indent over the tree. commentflow had not been run since the comments it now rewraps were written, including those added in #12, so this is its output plus the blank line it puts before a comment that follows code.

Every changed line is a comment or blank: a filter over git diff -U0 that drops comment and blank lines leaves nothing, and each blank-line hunk was checked in context to be outside any string literal. make check (280 unit and 11 stdio tests), cargo clippy --all-targets --all-features -- -D warnings, cargo fmt --check, bash -n, and shellcheck on the changed CI script all pass.


Summary by cubic

Runs make indent to rewrap comments that commentflow had not touched since they were written, including those added in the note-update change.

Every changed line is a comment or blank line; no behavior changes. make check, cargo clippy --all-targets --all-features -- -D warnings, cargo fmt --check, bash -n, and shellcheck all pass.

Written for commit 7506a54. Summary will update on new commits.

Review in cubic

commentflow had not run over the tree since the comments it now
rewraps were written, including the ones the note-update change just
added. This is its output, plus the blank line it puts before a comment
that follows code. Every changed line is a comment or blank, so nothing
the compiler or the shell reads is different.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 16 files

Re-trigger cubic

@jserv
jserv merged commit bbee77b into main Oct 5, 2026
11 checks passed
@jserv
jserv deleted the reflow branch October 5, 2026 07:54
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.

1 participant