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

Samples documentation fixes and enhanchements #905

Merged
merged 23 commits into from
Feb 26, 2024

Conversation

SaschaWillems
Copy link
Collaborator

Description

I went through all of the outstanding issues with regards to the samples documentation generated for https://docs.vulkan.org and hopefully fixed all open problems with this PR. Aside from fixing things, this PR also improves documentation in a few spots. Once merged documentation for our samples should be perfect now.

Bugfixes:

  • Fixed ~30 broken links, both to documents and images (antora build is now 100% clean for me)
  • Added in some referenced documents that were simply not present

Enhanchements:

  • Split the samples list into one file per folder (api, performance, extension, etc.), so that every sub folder in the docs site now shows the list of samples for this exact folder
  • The samples overview listing is now created by include the above files, so it looks just as a before
  • Added links to a few documents in the navigation that were there but not accessible through it yet

Fixes #903
Fixes #898
Fixes #824
Fixes #801

Note: This is a pure documentation fix

General Checklist:

Please ensure the following points are checked:

  • My code follows the coding style
  • I have reviewed file licenses
  • [n/a] I have commented any added functions (in line with Doxygen)
  • [n/a] I have commented any code that could be hard to understand
  • [n/a] My changes do not add any new compiler warnings
  • [n/a] My changes do not add any new validation layer errors or warnings
  • [n/a] I have used existing framework/helper functions where possible
  • [n/a] My changes do not add any regressions
  • [n/a] I have tested every sample to ensure everything runs correctly
  • [n/a] This PR describes the scope and expected impact of the changes I am making

List of examples in subfolder readme
This is then included in tioplevel sample listing instead
Paths need to be adjusted via attributes to make it work for both
Makes xrefs works both on the separate sample list as well as the page including it
Refs #898
Didn't work in github either, and makes no sense
People will see that template when they actually do a PR against the repo
Fix linkage for framework core
PRs are created via github anyway, and this automatically uses the PR template
No need to mention it explicitly
@SaschaWillems SaschaWillems added the documentation Improvements or additions to documentation label Feb 10, 2024
Copy link
Collaborator

@tomadamatkinson tomadamatkinson left a comment

Choose a reason for hiding this comment

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

Mega! Thanks @SaschaWillems for putting in the graft on this one

Is there a quick script we could right to make sure all the links work? Looks like all the paths are relative so we could make a small CI task to make life easier in the future?

Happy to help here

@SaschaWillems
Copy link
Collaborator Author

SaschaWillems commented Feb 10, 2024

Running antora to build the docs site will output errors. If you pipe that output you get JSON that can easily be parsed. Though I'm not sure if we can do this in this repo alone, as the samples are only a part of the whole docs repo. So if at all, we should do it in the doc site repo. Though I think Jon does that anyway.

Adding it to this repo for the CI would probably make life hard for contributors, as you'd need to know how to build an antpra project and how to fix things.

@tomadamatkinson
Copy link
Collaborator

Makes sense, it's low priority if we did pursue it. Jsut thought we might be able to whip something up in 30 minutes which might help save yourself a bit of effort in the long run

Happy to stay as is for now 😃

gary-sweet
gary-sweet previously approved these changes Feb 14, 2024
samples/README.adoc Outdated Show resolved Hide resolved
gary-sweet
gary-sweet previously approved these changes Feb 23, 2024
# Conflicts:
#	antora/modules/ROOT/nav.adoc
#	samples/README.adoc
@SaschaWillems
Copy link
Collaborator Author

@marty-johnson59 : Would be great of we could merge this on the next samples/doc call before we merge any other new sample PRs to avoid merge conflicts.

@marty-johnson59 marty-johnson59 merged commit 6683c7f into main Feb 26, 2024
24 checks passed
asuessenbach pushed a commit to asuessenbach/Vulkan-Samples that referenced this pull request Feb 27, 2024
* Include related extension list instead of linking to it (link was broken anyway)

* Different content depending on generator

* Reworking things

List of examples in subfolder readme
This is then included in tioplevel sample listing instead
Paths need to be adjusted via attributes to make it work for both

* Fix links to ray tracing samples KhronosGroup#903

* Fix xref paths

Makes xrefs works both on the separate sample list as well as the page including it
Refs KhronosGroup#898

* Fix links to ray tracing samples

Refs KhronosGroup#903

* Add samples listing to docs navigation

* Move api samples listing into separate readme

Include in main listing

* Added readme for general samples folder

Include in main listing

* Add samples listing to docs navigation

* Move tooling and performance samples listing into separate readme

Include in main listing

* Update pull request template

* Remove link to pull request template

Didn't work in github either, and makes no sense
People will see that template when they actually do a PR against the repo

* Add link to framework components

Fix linkage for framework core

* Fix link to debug utils sample

* Add a stub readme for the docs folder

* Fix broken links

Refs KhronosGroup#824

* Fix broken image links

Refs KhronosGroup#824

* Removed unnecessary paragraph

PRs are created via github anyway, and this automatically uses the PR template
No need to mention it explicitly

* Fix copyright

* Fix copyright (again...)

* Fix typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
5 participants