-
Notifications
You must be signed in to change notification settings - Fork 323
Contributing
See Adding Tests page for technical details on how to implement new test cases.
Vulkan CTS repository has currently two main branches: vulkan-cts-1.0
(default) and vulkan-cts-1.0-dev
.
This is the release branch for Vulkan 1.0 CTS. Only bugfixes are accepted in this branch.
This branch contains latest in-development version of Vulkan CTS. It is kept up-to-date with upstream dEQP project as well. All new tests and framework improvements are made in this branch.
dEQP is written in "traditional" C++ 98/03. Use of C++11 or newer is not currently allowed.
dEQP coding conventions are described in framework/delibs/coding_guidelines/de-coding-guidelines.html
See code under framework/delibs/decpp
for more examples on desired style.
Pull requests for new tests or features should target vulkan-cts-1.0-dev
, and bugfixes vulkan-cts-1.0
.
To keep git history meaningful, merge requests should generally contain just one, or a few atomic commits, meaning that all intermediate steps build & function correctly. When you need to update an existing pull request (to address feedback for example), please do so by modifying the existing commit(s) rather than adding additional commits on top.
To update one commit pull request, you can do:
git checkout <pull request branch>
# edit files
git add <modified files>
git commit --amend
git push --force origin <merge request branch>
--amend
flag modifies topmost commit instead of creating a new one.