[bdice]

This PR re-enables pre-commit checks for normalizing whitespace (trimming trailing spaces) that I disabled in #471.

These pre-commit hooks existed prior to the GitHub Actions migration but they were not being enforced by CI style checks. I disabled them in #471 because enforcing them would have made the PR diff large.

The primary change is in .pre-commit-config.yaml, and everything else was automatically fixed by pre-commit.

# TODO: re-enable this after GitHub Actions migration, it currently makes a
# large number of changes that shouldn't go in the GitHub Actions PR.
#- repo: https://github.com/pre-commit/pre-commit-hooks
#  rev: v4.4.0
#  hooks:
#    - id: trailing-whitespace
#    - id: end-of-file-fixer
#    - id: debug-statements

The base branch was changed.

[jakirkham]

Issue ( #484 ) has been addressed in branch-23.04. Have updated the PR branch here to merge in those changes

Also added labels to fix the label checker

[grlee77]

The changes look good. Looks like the CI failure is probably an unrelated network issue (failed download). I restarted the failed jobs to see if it goes through on the second round.

[grlee77]

Thanks, @bdice

[jakirkham]

Toggling for CI

[jakirkham]

@gpucibot merge

[rapids-bot]

@jakirkham, the @gpucibot merge command has been replaced with /merge.

Please re-comment this PR with /merge and use this new command in the future.

[jakirkham]

/merge

[jakirkham]

Think we want to keep this line. Otherwise the patch doesn't apply

[jakirkham]

Reverted in commit ( 43606cf )

[jakirkham]

Looks like the style check doesn't like that change 😂

If you are seeing this message in CI, reproduce locally with: `pre-commit run --all-files`.
To run `pre-commit` as part of git workflow, use `pre-commit install`.
All changes made by hooks:
diff --git a/cpp/cmake/deps/boost-header-only.patch b/cpp/cmake/deps/boost-header-only.patch
index e765790..96c0078 100644
--- a/cpp/cmake/deps/boost-header-only.patch
+++ b/cpp/cmake/deps/boost-header-only.patch
@@ -14,4 +14,3 @@ index 7da31f7..5816de7 100644
 +   //BOOST_ASSERT(0 == ((std::size_t)ret % ::boost::container::dtl::alignment_of<T>::value));
     return static_cast<T*>(ret);
  }
- 

Maybe we can skip this file during style checking?

[jakirkham]

Thanks all! 🙏