diff --git a/src/content/docs/Products/SplashKit/06-peer-review.mdx b/src/content/docs/Products/SplashKit/06-peer-review.mdx index e6f17e38..4ea59957 100644 --- a/src/content/docs/Products/SplashKit/06-peer-review.mdx +++ b/src/content/docs/Products/SplashKit/06-peer-review.mdx @@ -201,7 +201,7 @@ these steps when reviewing a PR in the SplashKit Starlight repository. - + ```md ## General Information @@ -241,9 +241,80 @@ these steps when reviewing a PR in the SplashKit Starlight repository. - [ ] Checklist Completion: Have all relevant checklist items been reviewed and completed? ``` - + + + + + ```md + ## Usage Example Review Checklist + + Use this checklist when reviewing SplashKit usage example Pull Requests to help maintain + consistency, quality, and clarity across contributions. + + ### Required Files + + - [ ] C++ implementation included (`.cpp`) + - [ ] Python implementation included (`.py`) + - [ ] C# OOP implementation included (`-oop.cs`) + - [ ] C# Top-Level implementation included (`-top-level.cs`) + - [ ] GIF demonstration included (`.gif`) + - [ ] Text description included (`.txt`) + + ### API Focus + + - [ ] The example clearly demonstrates the intended SplashKit API/function. + - [ ] The example focuses primarily on one SplashKit API/function. + - [ ] SplashKit functions are used instead of language-specific alternatives where possible. + - [ ] Additional functionality does not distract from the API being demonstrated. + - [ ] The example is simple and understandable for new SplashKit users. + + ### Cross-Language Consistency + + - [ ] C++, Python, C# OOP, and C# Top-Level demonstrate the same core behaviour. + - [ ] Equivalent SplashKit APIs are used consistently across supported languages. + - [ ] Comments, terminology, and functionality are reasonably consistent across implementations. + - [ ] File names follow the existing SplashKit usage example naming convention. + + ### Code Quality + + - [ ] The code is clear, readable, and easy to follow. + - [ ] Variable names are meaningful and consistent. + - [ ] Comments are included where they improve understanding. + - [ ] Unnecessary complexity and unrelated functionality have been avoided. + + ### Testing + + - [ ] The example has been tested and runs successfully. + - [ ] The C++ implementation compiles and runs successfully. + - [ ] The Python implementation runs successfully. + - [ ] The C# implementations use the appropriate SplashKit APIs and syntax. + - [ ] The demonstrated behaviour matches the purpose of the usage example. + + ### GIF and Description + + - [ ] The GIF clearly demonstrates the functionality of the example. + - [ ] The GIF matches the submitted implementation. + - [ ] The `.txt` description clearly and concisely describes the example. + + ### Repository Cleanliness + + - [ ] No compiled executables or build artefacts are included. + - [ ] No `.dSYM` directories are included. + - [ ] No log or temporary files are included. + - [ ] No unrelated files or changes are included in the Pull Request. + + ### Final Review + + - [ ] All required usage example files are present. + - [ ] The example remains focused on the intended SplashKit API. + - [ ] The implementations are consistent across supported languages. + - [ ] Previous review feedback has been addressed, where applicable. + - [ ] The Pull Request is ready for approval. + ``` + + - + #### Splashkit Review Prompts - **Type of Change**: Does this Pull Request correctly identify the type of change (bug fix, new