Skip to content

Remove all trailing whitespace in .c files#3216

Open
seanm wants to merge 2 commits intoUnidata:mainfrom
seanm:editorconfig2
Open

Remove all trailing whitespace in .c files#3216
seanm wants to merge 2 commits intoUnidata:mainfrom
seanm:editorconfig2

Conversation

@seanm
Copy link
Contributor

@seanm seanm commented Dec 2, 2025

No description provided.

seanm added 2 commits December 2, 2025 17:10
Done with:

find . -name '*.c' -exec perl -pi -e 's/[ \t]+$//' {} +
@WardF
Copy link
Member

WardF commented Dec 4, 2025

This touches a ton of files, so I'm going to wait until I've gotten to some of the other higher-priority PR's and issues.

@WardF WardF added this to the future milestone Dec 4, 2025
@WardF WardF self-assigned this Dec 4, 2025
@seanm
Copy link
Contributor Author

seanm commented Dec 4, 2025

Like I said in the other issue, this is more of a demo. Instead of actually reviewing this PR, you could just do the same perl command yourself and commit the result. That way you don't have to trust that I didn't slip anything sneaky in here.

@edhartnett
Copy link
Contributor

edhartnett commented Feb 18, 2026

I suggest this PR be closed without being merged. We all agree that this needs to be addressed piecemeal. I've started the ball rolling with #3271 . Thank you @seanm for your commitment to quality and your contributions to the netCDF project!

@seanm
Copy link
Contributor Author

seanm commented Feb 19, 2026

I suggest this PR be closed without being merged.

Yup, it was not really indented to be merged.

Thanks for taking up this work.

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.

3 participants