Skip to content
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

[SYCL][DOC] Move experimental extension docs #5436

Merged
merged 2 commits into from
Feb 1, 2022

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Jan 31, 2022

Move the experimental extensions to the
"sycl/doc/extensions/experimental" directory, and add a README
explaining the content of that directory. Each extension document has
also been renamed to match the name of its feature-test macro.

Links to these extensions have been removed from the table in
"sycl/doc/extensions/README.md" because we want to avoid the
possibility of that table getting out-of-date with the directory
contents. Instead, users can see the list of experimental extensions
by listing the content of the directory.

Move the experimental extensions to the
"sycl/doc/extensions/experimental" directory, and add a README
explaining the content of that directory.  Each extension document has
also been renamed to match the name of its feature-test macro.

Links to these extensions have been removed from the table in
"sycl/doc/extensions/README.md" because we want to avoid the
possibility of that table getting out-of-date with the directory
contents.  Instead, users can see the list of experimental extensions
by listing the content of the directory.
@gmlueck gmlueck requested a review from a team January 31, 2022 21:34
@gmlueck gmlueck requested review from a team as code owners January 31, 2022 21:34
pvchupin
pvchupin previously approved these changes Jan 31, 2022
kbobrovs
kbobrovs previously approved these changes Feb 1, 2022
@gmlueck gmlueck dismissed stale reviews from kbobrovs and pvchupin via 32f7aab February 1, 2022 16:29
@gmlueck
Copy link
Contributor Author

gmlueck commented Feb 1, 2022

@bader: I resolved conflicts, which dismissed approvals from @kbobrovs and @pvchupin. Can this be merged after checks complete, or do you need an approval also from @intel/dpcpp-specification-reviewers ?

@bader
Copy link
Contributor

bader commented Feb 1, 2022

I can do either way. Technically to enable merge button Required CI checks must pass and code owners approve. I have additional privileges for this this repository and can merge even if some requirements are not satisfied.
Let me know what you'd like to me to do.

@bader bader merged commit eae965c into intel:sycl Feb 1, 2022
@gmlueck gmlueck deleted the gmlueck/mv-experimental-extensions branch February 1, 2022 17:04
@lsatanov
Copy link
Contributor

lsatanov commented Feb 6, 2022

Hi. I've noticed that some public doc links are not working after this change, e.g., like those found in esimd doc page, see the "specification" link. Not sure who is responsible for those. Just FYI.

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