Skip to content

[SYCL] Enable documentation generation with Doxygen #878

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

Merged
merged 2 commits into from
Dec 2, 2019

Conversation

alexbatashev
Copy link
Contributor

Add doxygen-sycl target and make it process source code with Doxygen to emit
documentation. Generated HTML files can be found in
/path/to/build/tools/sycl/doc/html

Signed-off-by: Alexander Batashev [email protected]

Add doxygen-sycl target and make it process source code with Doxygen to emit
documentation. Generated HTML files can be found in
/path/to/build/tools/sycl/doc/html

Signed-off-by: Alexander Batashev <[email protected]>
@vladimirlaz vladimirlaz self-requested a review November 28, 2019 09:08
@bader bader dismissed vladimirlaz’s stale review November 28, 2019 09:37

Dismiss @vladimirlaz review for him :-).

Signed-off-by: Alexander Batashev <[email protected]>
@bader bader requested review from AGindinson and MrSidims November 29, 2019 16:29
@bader
Copy link
Contributor

bader commented Dec 2, 2019

@AGindinson, @MrSidims, any other comments? If no, could you approve, please?

@MrSidims
Copy link
Contributor

MrSidims commented Dec 2, 2019

@bader in general it's LGTM. But TBH it's hard to go through all the config and imagine what the result will be.

@AGindinson
Copy link
Contributor

@bader, I'm fine with @alexbatashev's resolution of my minor comments. I'd be taking on too much approving a PR I have little understanding of.

@bader bader merged commit de418d6 into intel:sycl Dec 2, 2019
@alexbatashev alexbatashev deleted the private/abatashe/docs_pt3 branch July 28, 2021 06:44
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