Add clean-up for whitespace to platform repo

[vogella]

@laeubi I would like to add a whitespace removal clean-up to this repo.

I think I saw a discussion between you and @HannesWell as few days ago how we can add a batch-clean-up to a repo. Can you point me to this so that I can see if I can add the whitespace cleanup?

[vogella]

Ah, found it: eclipse-equinox/equinox#936 (comment)

[jbampton]

Hell @vogella I would like to work on this issue.

My suggestion is to use pre-commit. It is a popular tool that uses Git hooks. From the pre-commit website:

"Git hook scripts are useful for identifying simple issues before submission to code review. We run our hooks on every commit to automatically point out issues in code such as missing semicolons, trailing whitespace, and debug statements. By pointing these issues out before code review, this allows a code reviewer to focus on the architecture of a change while not wasting time with trivial style nitpicks."

So pre-commit can run on the command line on git commit or with the pre-commit command. Also we can also add pre-commit to the GitHub CI with actions. I have done this on many repos.

https://pre-commit.com/

https://github.com/pre-commit/pre-commit

There are many hooks for pre-commit and some are listed here:

https://pre-commit.com/hooks.html

Also in terms of this issue for whitespace there are some official pre-commit hooks that can help.

All the official hooks are here:

https://github.com/pre-commit/pre-commit-hooks

And for whitespace there is:

• end-of-file-fixer: Makes sure files end in a newline and only a newline
• mixed-line-ending: Replaces or checks mixed line ending.
• trailing-whitespace: trims trailing whitespace

https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#end-of-file-fixer
https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#mixed-line-ending
https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#trailing-whitespace

So you can run lots of different linters and fixers with pre-commit including:

• actionlint
• codespell
• markdownlint
• markdown-link-check
• oxipng
• shellcheck
• yamllint

and more.

[vogella]

@jbampton I think the solution from @laeubi by using the related JDT clean-up is simpler. Did you check eclipse-equinox/equinox#936 (comment)?

[fedejeanne]

So something like eclipse-platform/eclipse.platform.releng.aggregator#3025 but using cleanup.remove_trailing_whitespaces (or cleanup.remove_trailing_whitespaces_all) instead?

[vogella]

@jbampton do you want to provide a PR for this to the aggregator repo using @fedejeanne suggestion? If yes, please use cleanup.remove_trailing_whitespaces_all to also include empty lines.

To see the clean-ups use right-click on project -> Source -> Clean-up -> Configure

[jbampton]

Hey @vogella thanks for listening to my proposal.

I just started working on another issue.

I will do more PRs for Eclipse in future.

Thanks.

[vogella]

Hey @vogella thanks for listening to my proposal.

Git pre-hooks can be nice but we already have a working solution re-using Eclipse JDT and adding one line to the existing solution seems more efficient than starting an alternative solution.

[HannesWell]

While I'm totally inclined in cleaning up the code, base I think we have to be careful with enabling such clean-ups that potentially create a lot of changes, similar to formatting the code to follow the Eclipse format or adding braces everywhere, as they could shadow previous changes in the git-blame. And unfortunately git-blame is still an important source of information.
I think that there is a solution to that problem by using git blame -ignore-rev, but this would then require more coordination.

[vogella]

While I'm totally inclined in cleaning up the code, base I think we have to be careful with enabling such clean-ups that potentially create a lot of changes, similar to formatting the code to follow the Eclipse format or adding braces everywhere, as they could shadow previous changes in the git-blame. And unfortunately git-blame is still an important source of information. I think that there is a solution to that problem by using git blame -ignore-rev, but this would then require more coordination.

Git has the ignore whitespace changes as well as EGit. A long time ago I ran the clean-up whitespaces on all repos I had access to as contributors and committer were complaining about about unnecessary white-space changes and enabled the save-actions for all plug-ins I found relevant. So this setting should ensure this standard is re-applied to the projects.

[merks]

Are you referring the "Show Revision" decorations in an editor or the Blame view on GitHub web page, or just to the button to ignore white space change when comparing state?

[HannesWell]

While I'm totally inclined in cleaning up the code, base I think we have to be careful with enabling such clean-ups that potentially create a lot of changes, similar to formatting the code to follow the Eclipse format or adding braces everywhere, as they could shadow previous changes in the git-blame. And unfortunately git-blame is still an important source of information. I think that there is a solution to that problem by using git blame -ignore-rev, but this would then require more coordination.

Git has the ignore whitespace changes as well as EGit.

I'm not referring to the Git Compare, where one can indeed ignore white-space, I'm referring to Git Blame shown on the left, where one can see the last commit that changed the corresponding line.

A long time ago I ran the clean-up whitespaces on all repos I had access to as contributors and committer were complaining about about unnecessary white-space changes and enabled the save-actions for all plug-ins I found relevant. So this setting should ensure this standard is re-applied to the projects.

If that's the case then all my previous comments can be disregarded, but it should be checked/estimated in advance how much the repos for which this workflow is in enabled are affected.