Skip to content

feat(clang-tidy): add clang tidy pr comments#334

Draft
yhisaki wants to merge 1 commit intomainfrom
clang-tidy-pr-comments
Draft

feat(clang-tidy): add clang tidy pr comments#334
yhisaki wants to merge 1 commit intomainfrom
clang-tidy-pr-comments

Conversation

@yhisaki
Copy link
Contributor

@yhisaki yhisaki commented Jan 29, 2025

Description

By using platisd/clang-tidy-pr-comments, we can annotate clang-tidy errors.

Tests performed

Not applicable.

Effects on system behavior

Not applicable.

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yhisaki yhisaki force-pushed the clang-tidy-pr-comments branch from 1c07a9c to 1d0a497 Compare February 17, 2025 02:44
@stale
Copy link

stale bot commented Apr 20, 2025

This pull request has been automatically marked as stale because it has not had recent activity.

@stale stale bot added the status:stale label Apr 20, 2025
@xmfcx
Copy link
Contributor

xmfcx commented Dec 24, 2025

@yhisaki -san would you like to update this PR?

@stale stale bot removed the status:stale label Dec 24, 2025
@xmfcx xmfcx force-pushed the clang-tidy-pr-comments branch from 1d0a497 to d8b8607 Compare December 24, 2025 18:20
@stale
Copy link

stale bot commented Feb 24, 2026

This pull request has been automatically marked as stale because it has not had recent activity.

@stale stale bot added the status:stale label Feb 24, 2026
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