mirror of
https://github.com/Bubberstation/Bubberstation.git
synced 2026-08-24 05:30:05 +01:00
[MIRROR] Adds a note in the guide for contributors and the PR template to explain what GBP is [MDB IGNORE] (#9219)
* Adds a note in the guide for contributors and the PR template to explain what GBP is * Fixing merge conflict Co-authored-by: Bond <58570888+TheBonded@users.noreply.github.com> Co-authored-by: GoldenAlpharex <58045821+GoldenAlpharex@users.noreply.github.com>
This commit is contained in:
co-authored by
Bond
GoldenAlpharex
parent
ccb19425d7
commit
ed6d58f68f
+18
-3
@@ -9,9 +9,10 @@
|
||||
3. [Issue Managers](#issue-managers)
|
||||
5. [Development Guides](#development-guides)
|
||||
6. [Pull Request Process](#pull-request-process)
|
||||
7. [Porting features/sprites/sounds/tools from other codebases](#porting-featuresspritessoundstools-from-other-codebases)
|
||||
8. [Banned content](#banned-content)
|
||||
9. [A word on Git](#a-word-on-git)
|
||||
7. [Good Boy Points](#good-boy-points)
|
||||
8. [Porting features/sprites/sounds/tools from other codebases](#porting-featuresspritessoundstools-from-other-codebases)
|
||||
9. [Banned content](#banned-content)
|
||||
10. [A word on Git](#a-word-on-git)
|
||||
|
||||
## Reporting Issues
|
||||
|
||||
@@ -136,6 +137,20 @@ There is no strict process when it comes to merging pull requests. Pull requests
|
||||
|
||||
* **Pull requests that include sprites must have in-game screenshots that were taken on your own test-server in the PR body.** For instance, if you're adding new clothes, a screenshot of those clothes being worn is expected in the PR body. Not every single direction needs to be displayed, but each icon that's actually being used in-game should be showcased.
|
||||
|
||||
## Good Boy Points
|
||||
|
||||
Each GitHub account has a score known as Good Boy Points, or GBP. This is a system we use to ensure that the codebase stays maintained and that contributors fix bugs as well as add features.
|
||||
|
||||
The GBP gain or loss for a PR depends on the type of changes the PR makes, represented by the tags assigned to the PR by the tgstation github bot or maintainers. Generally speaking, fixing bugs, updating sprites, or improving maps increases your GBP score, while adding mechanics, or rebalancing things will cost you GBP.
|
||||
|
||||
The GBP change of a PR is the sum of greatest positive and lowest negative values it has. For example, a PR that has tags worth +10, +4, -1, -7, will net 3 GBP (10 - 7).
|
||||
|
||||
Negative GBP increases the likelihood of a maintainer closing your PR. With that chance being higher the lower your GBP is. Be sure to use the proper tags in the changelog to prevent unnecessary GBP loss. Maintainers reserve the right to change tags as they deem appropriate.
|
||||
|
||||
There is no benefit to having a higher positive GBP score, since GBP only comes into consideration when it is negative.
|
||||
|
||||
You can see each tag and their GBP values [Here](https://github.com/tgstation/tgstation/blob/master/.github/gbp.toml).
|
||||
|
||||
## Porting features/sprites/sounds/tools from other codebases
|
||||
|
||||
If you are porting features/tools from other codebases, you must give them credit where it's due. Typically, crediting them in your pull request and the changelog is the recommended way of doing it. Take note of what license they use though, porting stuff from AGPLv3 and GPLv3 codebases are allowed.
|
||||
|
||||
@@ -11,7 +11,7 @@
|
||||
|
||||
## Changelog
|
||||
|
||||
<!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. -->
|
||||
<!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. -->
|
||||
|
||||
:cl:
|
||||
add: Added new mechanics or gameplay changes
|
||||
|
||||
Reference in New Issue
Block a user