Skip to content

[hipDNN] Fix clang tidy path detection#3367

Merged
SamuelReeder merged 2 commits into
developfrom
users/sareeder/clang-tidy-fix
Dec 15, 2025
Merged

[hipDNN] Fix clang tidy path detection#3367
SamuelReeder merged 2 commits into
developfrom
users/sareeder/clang-tidy-fix

Conversation

@SamuelReeder
Copy link
Copy Markdown
Contributor

@SamuelReeder SamuelReeder commented Dec 14, 2025

Motivation

GitHub Actions isn't compatible with curly brace regex, so switched to list each file extensions on new lines.

Test PR to show it triggers on an applicable file change: #3374

Submission Checklist

@SamuelReeder SamuelReeder force-pushed the users/sareeder/clang-tidy-fix branch from 9116024 to 70a59bc Compare December 15, 2025 05:18
@github-actions github-actions Bot added the project: none Does not target any component label Dec 15, 2025
@SamuelReeder SamuelReeder marked this pull request as ready for review December 15, 2025 15:27
@SamuelReeder SamuelReeder requested a review from a team as a code owner December 15, 2025 15:27
Copy link
Copy Markdown
Contributor

@CMiservaAMD CMiservaAMD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Comment thread .github/workflows/hipdnn-clang-tidy.yml Outdated
@SamuelReeder SamuelReeder merged commit a1812cb into develop Dec 15, 2025
14 checks passed
@SamuelReeder SamuelReeder deleted the users/sareeder/clang-tidy-fix branch December 15, 2025 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants