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.

Sign up for GitHub

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

Jump to bottom

Restructured readme, moved tutorials to sample readme #207

Merged
merged 4 commits into from Nov 23, 2020
Merged

Restructured readme, moved tutorials to sample readme #207

merged 4 commits into from Nov 23, 2020

Conversation

SaschaWillems
Copy link
Contributor

Description

This PR moves the tutorials from the main repo readme to the samples readme. Instead of a single listing of all available tutorials, available tutorials are now associated with their accompanying sample.

General Checklist:

Please ensure the following points are checked:

  • [n/a] 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 build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
  • [n/a] My changes do not add any regressions
  • [n/a] I have tested every sample to ensure everything runs correctly
  • This PR describes the scope and expected impact of the changes I am making

r-potter
r-potter previously approved these changes Nov 9, 2020
View reviewed changes
aurolou
aurolou previously approved these changes Nov 9, 2020
View reviewed changes
@SaschaWillems SaschaWillems dismissed stale reviews from aurolou and r-potter via aa9df5a November 16, 2020 06:58
@marty-johnson59 marty-johnson59 merged commit b4fe3ad into KhronosGroup:master Nov 23, 2020
ShaoyuanLi pushed a commit to ShaoyuanLi/Vulkan-Samples that referenced this pull request Dec 11, 2020
@SaschaWillems @ShaoyuanLi
Restructured readme, moved tutorials to sample readme (KhronosGroup#207)
478ac81 
* Reworked main readme, moved tutorials to sample readme

* Reworked main readme, moved tutorials to sample readme

* Removed tutorials links

* Added "General information" section to TOC
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gary-sweet gary-sweet gary-sweet approved these changes

@JoseEmilio-ARM JoseEmilio-ARM JoseEmilio-ARM approved these changes

@carlosadcq carlosadcq carlosadcq approved these changes

@AlejandroCosin AlejandroCosin Awaiting requested review from AlejandroCosin

@mattruschnv mattruschnv Awaiting requested review from mattruschnv

@wasimabbas-arm wasimabbas-arm Awaiting requested review from wasimabbas-arm

@aurolou aurolou Awaiting requested review from aurolou

@r-potter r-potter Awaiting requested review from r-potter

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@SaschaWillems @r-potter @aurolou @gary-sweet @JoseEmilio-ARM @carlosadcq @marty-johnson59