Skip to content

Conversation

AlexeySotkin
Copy link
Contributor

Signed-off-by: Alexey Sotkin alexey.sotkin@intel.com

@AlexeySotkin AlexeySotkin requested a review from MrSidims August 4, 2021 18:12
MrSidims
MrSidims previously approved these changes Aug 5, 2021
@AlexeySotkin AlexeySotkin marked this pull request as ready for review August 11, 2021 14:09
AGindinson
AGindinson previously approved these changes Aug 17, 2021
hchilama
hchilama previously approved these changes Aug 17, 2021
Copy link
Contributor

@hchilama hchilama left a comment

Choose a reason for hiding this comment

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

LGTM

hchilama
hchilama previously approved these changes Aug 19, 2021
Signed-off-by: Alexey Sotkin <alexey.sotkin@intel.com>
@AlexeySotkin
Copy link
Contributor Author

@bader, I think this PR is ready for merge. Please take a look.

@bader
Copy link
Contributor

bader commented Aug 27, 2021

@bader, I think this PR is ready for merge. Please take a look.

I'd like to fix post-commit testing before merging patches. Please, address pre-commit issue for #4411 as soon as possible.

@AlexeySotkin
Copy link
Contributor Author

@bader, I think this PR is ready for merge. Please take a look.

I'd like to fix post-commit testing before merging patches. Please, address pre-commit issue for #4411 as soon as possible.

@bader the issue for #4411 is resolved.

@bader bader merged commit a418e1c into intel:sycl Aug 30, 2021
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.

6 participants