From 3a735045c943bfb6286d1b7d8d24b086d1da5c87 Mon Sep 17 00:00:00 2001 From: DEATHB4DEFEAT <77995199+DEATHB4DEFEAT@users.noreply.github.com> Date: Wed, 25 Oct 2023 20:40:45 -0700 Subject: [PATCH] PR and Issue Improvements (#1) uhhhh how do I use this? we haven't guidelines yet! --- .github/CODEOWNERS | 70 ++++++-------------------- .github/ISSUE_TEMPLATE/issue_report.md | 28 ++++++++--- .github/PULL_REQUEST_TEMPLATE.md | 49 +++++++----------- .github/labeler.yml | 26 +++++++--- 4 files changed, 74 insertions(+), 99 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 1adc0b803d..6a40012b4c 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,65 +1,27 @@ # Last match in file takes precedence. -# Sorting by path instead of by who added it one day :( -# this isn't how codeowners rules work pls read the first comment instead of trying to force a sorting order -/Resources/ConfigPresets/WizardsDen/ @Chief-Engineer +/.* @DEATHB4DEFEAT -# Moony's Gargantuan List Of Things She Cares About, or MGLOTSCA for short. -# You need to add your name to these entries, not make a new one, if you care about them. -/Content.*/Toolshed/ @moonheart08 -**/Toolshed/** @moonheart08 -*Command.cs @moonheart08 -/Content.*/Administration/ @moonheart08 @DrSmugleaf @Chief-Engineer -/Content.*/Station/ @moonheart08 -/Content.*/Maps/ @moonheart08 -/Content.*/GameTicking/ @moonheart08 @EmoGarbage404 -/Resources/ServerInfo/ @moonheart08 @Chief-Engineer -/Resources/ServerInfo/Guidebook/ @moonheart08 @EmoGarbage404 -/Resources/engineCommandPerms.yml @moonheart08 @Chief-Engineer -/Resources/clientCommandPerms.yml @moonheart08 @Chief-Engineer -/Resources/Prototypes/Maps/ @Emisse +/Content.*/ @DEATHB4DEFEAT +/Content.*/SimpleStation14/ @Pspritechologist @DEATHB4DEFEAT -/Resources/Prototypes/Body/ @DrSmugleaf # suffering -/Resources/Prototypes/Entities/Mobs/Player/ @DrSmugleaf -/Resources/Prototypes/Entities/Mobs/Species/ @DrSmugleaf -/Content.*/Body/ @DrSmugleaf -/Content.YAMLLinter @DrSmugleaf -/Content.*/Verbs/ @ElectroJr -/Content.Client/ContextMenu/ @ElectroJr -/Content.Shared/Damage/ @ElectroJr @DrSmugleaf -/Content.Shared/Containers/ItemSlot/ @ElectroJr -/Content.*/PowerCell/ @ElectroJr -/Content.*/Doors/ @ElectroJr -/Content.*/Interaction/ @ElectroJr -/Content.*/Hands/ @ElectroJr -/Content.*/Actions/ @ElectroJr -/Content.*/Explosion/ @ElectroJr +/Resources/ @DEATHB4DEFEAT -/Content.*/Anomaly/ @EmoGarbage404 -/Content.*/Lathe/ @EmoGarbage404 -/Content.*/Materials/ @EmoGarbage404 -/Content.*/Mech/ @EmoGarbage404 -/Content.*/Research/ @EmoGarbage404 -/Content.*/Stack/ @EmoGarbage404 -/Content.*/Xenoarchaeology/ @EmoGarbage404 -/Content.*/Zombies/ @EmoGarbage404 -/Resources/Prototypes/Entities/Structures/Specific/anomalies.yml @EmoGarbage404 -/Resources/Prototypes/Research/ @EmoGarbage404 +/Resources/Audio/SimpleStation14 @Pspritechologist @DEATHB4DEFEAT -/Content.*/Forensics/ @ficcialfaint +/Resources/Maps/ @Pspritechologist @DEATHB4DEFEAT -# SKREEEE -/Content.*.Database/ @PJB3005 @DrSmugleaf -/Content.Shared.Database/Log*.cs @PJB3005 @DrSmugleaf @Chief-Engineer -/Pow3r/ @PJB3005 -/Content.Server/Power/Pow3r/ @PJB3005 +/Resources/Prototypes/Maps @Pspritechologist @DEATHB4DEFEAT +/Resources/Prototypes/SimpleStation14/ @Pspritechologist @DEATHB4DEFEAT -# notafet -/Content.*/Atmos/ @Partmedia -/Content.*/Botany/ @Partmedia +/Resources/Textures/SimpleStation14/ @Pspritechologist @DEATHB4DEFEAT -#Jezi -/Content.*/Medical @Jezithyr -/Content.*/Body @Jezithyr +*.ftl @Pspritechologist @DEATHB4DEFEAT + + +/Scripts/ @DEATHB4DEFEAT +/Scripts/sh/ @AriaMoradi + +/sloth.txt @Pspritechologist @DEATHB4DEFEAT @TemporalPrdx diff --git a/.github/ISSUE_TEMPLATE/issue_report.md b/.github/ISSUE_TEMPLATE/issue_report.md index 9bc805b74e..f18221e055 100644 --- a/.github/ISSUE_TEMPLATE/issue_report.md +++ b/.github/ISSUE_TEMPLATE/issue_report.md @@ -7,14 +7,26 @@ assignees: '' --- -## Description - +# Description + -**Reproduction** - +# Reproduction + -**Screenshots** - +# Screenshots + -**Additional context** - +# Additional context + diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index a397604185..71e6cc7125 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,43 +1,32 @@ - - +# Description -## About the PR - +Description. -## Why / Balance - +--- -## Technical details - +# Tasks -## Media - +

Media

+

-- [ ] I have added screenshots/videos to this PR showcasing its changes ingame, **or** this PR does not require an ingame showcase + -## Breaking changes - +

+
-**Changelog** - +--- + +# Changelog + + - +- remove: Removed fun! diff --git a/.github/labeler.yml b/.github/labeler.yml index b088f22943..32936a6180 100644 --- a/.github/labeler.yml +++ b/.github/labeler.yml @@ -1,12 +1,24 @@ -"Changes: Sprites": -- '**/*.rsi/*.png' +"Changes: C#": + - "**/*.cs" + +"Changes: Documentation": + - "**/*.xml" + - "**/*.md" + +"Changes: Localization": +- 'Resources/Locale/**/*.ftl' "Changes: Map": -- 'Resources/Maps/*.yml' -- 'Resources/Prototypes/Maps/*.yml' + - "Resources/Maps/**/*.yml" + - "Resources/Prototypes/Maps/**/*.yml" + +"Changes: Sprite": + - "**/*.rsi/*.png" + - "**/*.rsi/*.json" "Changes: UI": -- '**/*.xaml*' + - "**/*.xaml*" -"No C#": -- all: ["!**/*.cs"] +"Changes: YML": + - any: ["**/*.yml"] + all: ["!Resources/Maps/**/*.yml", "!Resources/Prototypes/Maps/**/*.yml"]