Skip to content

WIP: added .editorconfig file, applied uncrustify to make indentation uniform - #12

Draft
seanm wants to merge 4 commits into
InsightSoftwareConsortium:masterfrom
seanm:editorconfig
Draft

WIP: added .editorconfig file, applied uncrustify to make indentation uniform#12
seanm wants to merge 4 commits into
InsightSoftwareConsortium:masterfrom
seanm:editorconfig

Conversation

@seanm

@seanm seanm commented Feb 13, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@hjmjohnson

Copy link
Copy Markdown
Member

@seanm If we are going to diverge from the main nifti_c lib, I would propose making as many changes as necessary in one commit.

I prefer to use clang format. I have had better luck with integration into my editor workflows. Additionally, we could keep in sync with ITK and reduce the overall maintenance headaches.

@seanm

seanm commented Feb 16, 2025

Copy link
Copy Markdown
Collaborator Author

I guess I thought at least making the indentation uniform would make it easier to work on the codebase, because currently it's a pain since touching different parts requires using different indentation, which my editor does not like.

I'd also like to use clang-format eventually.

I could give it another try, but even after using whatstyle clang-format was making many massive changes...

@seanm
seanm marked this pull request as draft July 31, 2026 19: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.

2 participants