Skip to content

Conversation

@yingcong-wu
Copy link
Contributor

@aarongreig aarongreig marked this pull request as ready for review October 11, 2024 10:03
@aarongreig aarongreig requested a review from a team as a code owner October 11, 2024 10:03
@aarongreig
Copy link
Contributor

@intel/llvm-gatekeepers please merge

@martygrant martygrant merged commit 35b926a into intel:sycl Oct 11, 2024
12 checks passed
@sarnex
Copy link
Contributor

sarnex commented Oct 11, 2024

@martygrant In the future we should probably make sure the commit title is finalized and matches our requirements when merging, this one seems a bit undescriptive, thanks!

@yingcong-wu yingcong-wu deleted the yc/0829_UR-pr-2028 branch October 28, 2024 02:43
@martygrant
Copy link
Contributor

@sarnex apologies I never saw your comment, are these guidelines documented somewhere?

@sarnex
Copy link
Contributor

sarnex commented Nov 4, 2024

@martygrant Np, I think it's here

@sarnex
Copy link
Contributor

sarnex commented Nov 4, 2024

@martygrant Sorry also here

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.

4 participants