-
Notifications
You must be signed in to change notification settings - Fork 663
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(ci): fix failure condition when modified files are empty #7704
chore(ci): fix failure condition when modified files are empty #7704
Conversation
Signed-off-by: Kotaro Yoshimoto <[email protected]>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Signed-off-by: Kotaro Yoshimoto <[email protected]>
@HansRobo |
Signed-off-by: GitHub <[email protected]>
Signed-off-by: Kotaro Yoshimoto <[email protected]>
Signed-off-by: Kotaro Yoshimoto <[email protected]>
This reverts commit 3b8bcb3.
Signed-off-by: Kotaro Yoshimoto <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really appreciate your quick work!! LGTM 👍
@HansRobo |
@veqcc I set DCO to pass manually. |
…arefoundation#7704) * chore(ci): fix failure condition when modified files are empty Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete unnecessary new line Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete lidar_centerpoint_tvm package for test Signed-off-by: GitHub <[email protected]> * chore(ci): fix workflow conditions Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore(ci): filter un-existing files in cppcheck-differential Signed-off-by: Kotaro Yoshimoto <[email protected]> * Revert "chore: delete lidar_centerpoint_tvm package for test" This reverts commit 3b8bcb3. * chore: fix pre-commit errors Signed-off-by: Kotaro Yoshimoto <[email protected]> --------- Signed-off-by: Kotaro Yoshimoto <[email protected]> Signed-off-by: GitHub <[email protected]> Signed-off-by: Simon Eisenmann <[email protected]>
…arefoundation#7704) * chore(ci): fix failure condition when modified files are empty Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete unnecessary new line Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete lidar_centerpoint_tvm package for test Signed-off-by: GitHub <[email protected]> * chore(ci): fix workflow conditions Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore(ci): filter un-existing files in cppcheck-differential Signed-off-by: Kotaro Yoshimoto <[email protected]> * Revert "chore: delete lidar_centerpoint_tvm package for test" This reverts commit 3b8bcb3. * chore: fix pre-commit errors Signed-off-by: Kotaro Yoshimoto <[email protected]> --------- Signed-off-by: Kotaro Yoshimoto <[email protected]> Signed-off-by: GitHub <[email protected]>
…arefoundation#7704) * chore(ci): fix failure condition when modified files are empty Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete unnecessary new line Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete lidar_centerpoint_tvm package for test Signed-off-by: GitHub <[email protected]> * chore(ci): fix workflow conditions Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore(ci): filter un-existing files in cppcheck-differential Signed-off-by: Kotaro Yoshimoto <[email protected]> * Revert "chore: delete lidar_centerpoint_tvm package for test" This reverts commit 3b8bcb3. * chore: fix pre-commit errors Signed-off-by: Kotaro Yoshimoto <[email protected]> --------- Signed-off-by: Kotaro Yoshimoto <[email protected]> Signed-off-by: GitHub <[email protected]> Signed-off-by: palas21 <[email protected]>
…arefoundation#7704) * chore(ci): fix failure condition when modified files are empty Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete unnecessary new line Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete lidar_centerpoint_tvm package for test Signed-off-by: GitHub <[email protected]> * chore(ci): fix workflow conditions Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore(ci): filter un-existing files in cppcheck-differential Signed-off-by: Kotaro Yoshimoto <[email protected]> * Revert "chore: delete lidar_centerpoint_tvm package for test" This reverts commit 3b8bcb3. * chore: fix pre-commit errors Signed-off-by: Kotaro Yoshimoto <[email protected]> --------- Signed-off-by: Kotaro Yoshimoto <[email protected]> Signed-off-by: GitHub <[email protected]>
* chore(ci): fix failure condition when modified files are empty Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete unnecessary new line Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore: delete lidar_centerpoint_tvm package for test Signed-off-by: GitHub <[email protected]> * chore(ci): fix workflow conditions Signed-off-by: Kotaro Yoshimoto <[email protected]> * chore(ci): filter un-existing files in cppcheck-differential Signed-off-by: Kotaro Yoshimoto <[email protected]> * Revert "chore: delete lidar_centerpoint_tvm package for test" This reverts commit 3b8bcb3. * chore: fix pre-commit errors Signed-off-by: Kotaro Yoshimoto <[email protected]> --------- Signed-off-by: Kotaro Yoshimoto <[email protected]> Signed-off-by: GitHub <[email protected]>
Description
Fix failure condition when modified files are empty in
cppcheck-differential
.Related links
A test due to empty modified files. (the PR deletes all the package files)
https://github.com/autowarefoundation/autoware.universe/actions/runs/9673888348/job/26688603324
How was this PR tested?
The workflow works well with diffs that includes deleted files only.
https://github.com/HansRobo/autoware.universe/actions/runs/9675016733/job/26691735752
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.