use shared action for clang-tidy #3109
Conversation
There was a problem hiding this comment.
Pull request overview
This PR refactors the clang-tidy step in the GitHub Actions workflow to use a shared, reusable action instead of inline bash commands.
Changes:
- Replaced inline bash script execution with a call to a shared action (
activeloopai/private-github-actions/.github/actions/clang-tidy@main) - Updated build directory from
builds/deeplake-pg-prodtobuilds/deeplake-pg-dev
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| echo -e "${GREEN}Clang-tidy analysis complete.${DEFAULT}" | ||
| with: | ||
| source_directory: cpp/deeplake_pg | ||
| build_directory: builds/deeplake-pg-dev |
There was a problem hiding this comment.
The build directory was changed from builds/deeplake-pg-prod to builds/deeplake-pg-dev. This appears to be unintentional as the PR description does not mention changing the build target. Verify this change is correct, or revert to builds/deeplake-pg-prod to maintain consistency with the previous configuration.
| build_directory: builds/deeplake-pg-dev | |
| build_directory: builds/deeplake-pg-prod |
|
No description provided.