-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL][Doc] Add FAQ section #1045
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
Conversation
This patch provides basic FAQ section with answers to the most frequently asked questions. The list of questions is not complete and will be extended by other patches. Co-authored-by: Artem Gindinson <[email protected]> Signed-off-by: Alexander Batashev <[email protected]>
Signed-off-by: Artem Gindinson <[email protected]>
4cc03aa
to
429da8d
Compare
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.
LGTM content-wise, just some style/grammar comments.
Please, limit lines to 80 characters.
Signed-off-by: Artem Gindinson <[email protected]>
Signed-off-by: Artem Gindinson <[email protected]>
Applied most of the suggestions, resolving the correspondent comments. |
Signed-off-by: Alexander Batashev <[email protected]>
Signed-off-by: Artem Gindinson <[email protected]>
Hopefully, a SYCL LIT failure won't block this PR... |
No, but I'd like to receive an approve from @pvchupin. |
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.
As discussed offline please consider transforming some of Q&A into documentation.
This patch provides basic FAQ section with answers to the most
frequently asked questions. The list of questions is not complete and
will be extended by other patches.
Co-authored-by: Artem Gindinson [email protected]
Signed-off-by: Alexander Batashev [email protected]