Skip to content

Conversation

@maarquitos14
Copy link
Contributor

Most likely the clang-format checker was down when #14575 ran CI, so a few changes non-clang-format-compliant made it in. Applying fixes to those.

Signed-off-by: Marcos Maronas <marcos.maronas@intel.com>
@maarquitos14 maarquitos14 requested review from a team and bader as code owners August 5, 2024 11:10
@maarquitos14
Copy link
Contributor Author

@aelovikov-intel FYI

marcos.patch Outdated
@@ -0,0 +1,92 @@
From 2b6e88d4d01fea816f525fafc71676603f7a8c53 Mon Sep 17 00:00:00 2001
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like this patch file snuck into your commit

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, thanks for catching this!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed :)

Signed-off-by: Marcos Maronas <marcos.maronas@intel.com>
Signed-off-by: Marcos Maronas <marcos.maronas@intel.com>
@maarquitos14
Copy link
Contributor Author

@bader Friendly ping.

@maarquitos14
Copy link
Contributor Author

maarquitos14 commented Aug 7, 2024

@mdtoguchi Friendly ping.

@bader bader requested a review from mdtoguchi August 7, 2024 14:19
@maarquitos14
Copy link
Contributor Author

@intel/llvm-gatekeepers this is ready to merge.

@ldrumm ldrumm merged commit 65ee744 into intel:sycl Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants