Skip to content

refactor(mpi): drop commented-out write_vtk call - #22

Merged
nikbott merged 2 commits into
devfrom
refactor/rm-commented-viz
Jul 7, 2026
Merged

refactor(mpi): drop commented-out write_vtk call#22
nikbott merged 2 commits into
devfrom
refactor/rm-commented-viz

Conversation

@nikbott

@nikbott nikbott commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Removes the dead commented-out viz::write_vtk(...) line in mpi/main.cpp flagged by CodeQL (cpp/commented-out-code) — the one open code-scanning alert. Viz remains available via mpi/viz.hpp when wanted.

🤖 Generated with Claude Code

nikbott and others added 2 commits July 7, 2026 09:31
main.cpp predates enforced clang-format and was never in a lint changed-set,
so it carried trailing whitespace and non-conforming line breaks. Pure
formatting, no behavior change — done separately so the follow-up removal is
reviewable on its own.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Dead commented-out code flagged by CodeQL (cpp/commented-out-code) — the one
open code-scanning alert. Viz remains available via mpi/viz.hpp when wanted.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nikbott
nikbott force-pushed the refactor/rm-commented-viz branch from 28b0853 to ac8a95f Compare July 7, 2026 12:32
@nikbott
nikbott merged commit 31176cf into dev Jul 7, 2026
9 checks passed
@nikbott
nikbott deleted the refactor/rm-commented-viz branch July 7, 2026 12:35
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