mirror of
https://github.com/spice2x/spice2x.github.io.git
synced 2026-08-02 06:40:42 -07:00
Compare commits
6 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 5bce47e487 | |||
| ccf2009726 | |||
| 0253cb84a7 | |||
| 4d7017bd07 | |||
| cb4bd6e80d | |||
| 7ed33e261e |
@@ -17,6 +17,8 @@ Additionally, there's a lot of code that get exercised on specific hardware - in
|
|||||||
|
|
||||||
Therefore, when making code changes, please be extremely careful about containing / scoping your changes. Make targeted bug fixes scoped to handful of game versions and hardware configuration. When adding new features, make it off by default, unless there is a really good reason to make it the default. If you make a new default, add an option that disables it so that users can opt out as needed.
|
Therefore, when making code changes, please be extremely careful about containing / scoping your changes. Make targeted bug fixes scoped to handful of game versions and hardware configuration. When adding new features, make it off by default, unless there is a really good reason to make it the default. If you make a new default, add an option that disables it so that users can opt out as needed.
|
||||||
|
|
||||||
|
Lastly, watch out for legacy OS compatibility. Currently, the minimum support floor is Windows 7. If you use any Windows API, make sure it's supported in Windows 7. If you need to use API that is not present in Windows 7, you must not directly link against it, otherwise spice will fail to launch on older OSes. Take a look at Windows 8 touch code (win8.cpp) for examples on how to discover OS APIs via pointers.
|
||||||
|
|
||||||
### Code quality requirements
|
### Code quality requirements
|
||||||
|
|
||||||
* Test for regressions, at least in and around the component you are modifying:
|
* Test for regressions, at least in and around the component you are modifying:
|
||||||
@@ -29,3 +31,4 @@ Therefore, when making code changes, please be extremely careful about containin
|
|||||||
* Do not make code changes in unrelated areas; i.e., do not run code linters and auto-formatters for parts of the code that you didn't modify.
|
* Do not make code changes in unrelated areas; i.e., do not run code linters and auto-formatters for parts of the code that you didn't modify.
|
||||||
* Note: there are no strict rules for code formatting, but please attempt to emulate the style around the code you are modifying.
|
* Note: there are no strict rules for code formatting, but please attempt to emulate the style around the code you are modifying.
|
||||||
* Try to submit smaller chunks of code, instead one gigantic patch. For example, don't submit a patch for "Improve feature XYZ"; instead, submit "Change how A works to prepare for feature XYZ" "Refactor B for feature XYZ" "Add feature B to enable feature XYZ".
|
* Try to submit smaller chunks of code, instead one gigantic patch. For example, don't submit a patch for "Improve feature XYZ"; instead, submit "Change how A works to prepare for feature XYZ" "Refactor B for feature XYZ" "Add feature B to enable feature XYZ".
|
||||||
|
* Write to the log for anything useful - it helps immensely with debugging post-mortem. At the same time though, avoid spamming the log for something trivial.
|
||||||
|
|||||||
@@ -35,7 +35,9 @@ Rules for filing a new issue or adding comments to existing issues in the tracke
|
|||||||
* Check the [known issues](https://github.com/spice2x/spice2x.github.io/wiki/Known-issues) page first before reporting a new issue.
|
* Check the [known issues](https://github.com/spice2x/spice2x.github.io/wiki/Known-issues) page first before reporting a new issue.
|
||||||
* Use the search function and see if there is an existing issue.
|
* Use the search function and see if there is an existing issue.
|
||||||
* This is not the place to obtain a guide or receive basic troubleshooting.
|
* This is not the place to obtain a guide or receive basic troubleshooting.
|
||||||
* **Do not link to external websites that distribute game data!**
|
* Don't file a bug demanding game XYZ to be supported.
|
||||||
|
* **Do not upload game data - any part of it, XML files included!**
|
||||||
|
* **Do not mention or link to external websites that distribute game data!**
|
||||||
* **Do not link to external websites that provide guides on how to run games!**
|
* **Do not link to external websites that provide guides on how to run games!**
|
||||||
|
|
||||||
New GitHub accounts are prevented from creating new issues to prevent spam. Maintainers of this project reserve the right to close or delete any issues that violate the rules above, or low effort issues.
|
New GitHub accounts are prevented from creating new issues to prevent spam. Maintainers of this project reserve the right to close or delete any issues that violate the rules above, or low effort issues.
|
||||||
|
|||||||
Reference in New Issue
Block a user