* Added more bugprone-* checks, added -DBUILD_surface_on_nurbs=ON to clang-tidy GitHub action
* Part A: Run new bugprone checks, excluding the apps subdirectories
* Fix dynamic_cast missing const
* Revert -exclude in GitHub action, add .clang-tidy in directories to exclude from checks
* Modify to avoid "no checks enabled" errors
* Repair directory-level .clang-tidy files
* Switch to use an empty Checks string
* Revert to Checks: '-*'
* Implement a minimal clang-tidy check in third-party code
* Another try to disable clang-tidy in third-party directories
* Fix forward decls flagged by clang-tidy
* Address review comments, run clang-format on whitelist
* Revert "Address review comments, run clang-format on whitelist"
This reverts commit 60a7ee4c54.
* Readdress review comments
* Use only YAML formatter on GitHub Action
* Remove redundant using statement per review
* Address Copilot code review issues
* Change to use const auto* with dynamic_cast
- release pipeline: fix git tag of final release (was previously e.g. pcl-1.13.0-rc0, is now pcl-1.13.0). Pre-releases/release-candidates are still pcl-1.13.0-rc1
- ubuntu-variety: update available compilers
- install libboost-system-dev. While this is installed implicitly by libboost-filesystem-dev, PCL needs the `system` module, so we should install it explicitly
- install libpcap-dev, an optional dependency of PCL
- update actions/checkout to v3 (see https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/)
* Ran clang-tidy on apps, apps_3d_rec_framework [PCL-5304]
* Ran clang-tidy on apps_cloud_composer through apps_point_cloud_editor
* Ran clang-tidy on benchmarks through global_tests
* Fixed issues flagged by clang-format [PCL-5304]
* Fixed remaining clang-format issue [PCL-5304]
* Added first set of optional modules to clang-tidy, and fixes
* Fixed clang-tidy issues from optional modules [PCL-5304]
* Added NOLINT to work around clang-tidy issue [PCL-5304]
* Omitted apps module to test effect on clang-tidy [PCL-5304]
* Enabled apps* modules for clang-tidy
* Switched to tidy non-apps* optional modules [PCL-5304]
This alternates the behavior of the stalebot in several ways:
* Issues can now be stale. We want the bot to chase down unconfirmed
issues abandoned by their authors. If the author has lost interest,
why would we care?
* Reduce timeout until issue/pr is considered stale to 1 month. One of
the purposes of marking issues/prs as stale is to poke their authors
to return and finish off the work. The longer we wait, the less the
chance they are still somewhat motivated.
* Stale issues are automatically closed after 7 days. If the author
did not reply to the poke, close the issue soon. It can be re-opened
anyway, if the author awakens.
* Only "needs: author reply" and "needs: more work" labels can become
stale. These indicate that an action from the author is needed.
