Update PR templates again (#164)

This commit is contained in:
Justin Edmund 2025-01-15 21:52:20 -08:00 committed by GitHub
parent 7b74105e63
commit bde8dffaa2
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 69 additions and 100 deletions

View file

@ -1,9 +1,5 @@
blank_issues_enabled: false Choose a PR template:
contact_links:
- name: Default Pull Request * [General]("?expand=1&template=.github/PULL_REQUEST_TEMPLATE/default.md"): For general changes like feature additions
url: .github/PULL_REQUEST_TEMPLATE/default.md and issue resolution
about: For general changes and updates * [Data Update]("?expand=1&template=.github/PULL_REQUEST_TEMPLATE/data_update.md"): For game data updates
- name: Data Update Pull Request
url: .github/PULL_REQUEST_TEMPLATE/data_update.md
about: For game data updates and modifications

View file

@ -1,68 +1,46 @@
name: Data Update # Data Update
description: For game data updates and modifications
body:
- type: markdown ## Summary
attributes:
value: | <!-- Describe what this data update includes -->
## Data Update <!-- Example: Adding new Valentines 2024 characters -->
- type: textarea
id: summary ## New Additions
attributes:
label: Summary <!-- Copy the list of newly added items from the Import Summary -->
description: Describe what this data update includes
placeholder: "Adding new Valentines 2024 characters" ##### Characters
validations: -
required: true
- type: textarea ##### Weapons
id: new-additions -
attributes:
label: New Additions ##### Summons
description: List new items being added, grouped by type -
value: |
##### Characters ## Modifications
-
##### Weapons <!-- Copy the list of modified items from the Import Summary -->
-
##### Summons ##### Characters
- -
- type: textarea
id: modifications ##### Weapons
attributes: -
label: Modifications
description: List existing items being modified ##### Summons
value: | -
##### Characters
- ## Checklist
##### Weapons
- - [ ] CSV files use the correct naming format (`YYYYMMDD-{type}-XXX.csv`)
##### Summons - [ ] CSV files are in the correct location (`db/seed/updates/`)
- - [ ] All required fields are filled out
- type: textarea - [ ] Dates use the correct format (`YYYY-MM-DD`)
id: csv-files - [ ] Boolean values are either `true` or `false`
attributes: - [ ] Arrays use the correct format (e.g., `{value1,value2}`)
label: CSV Files Added - [ ] Ran import in test mode (`bin/rails granblue:import_data TEST=true`)
description: List all CSV files included in this update
value: | ## Test Results
- [ ] `YYYYMMDD-characters-XXX.csv`
- [ ] `YYYYMMDD-weapons-XXX.csv` <!-- Paste the completion section from running the import in test mode -->
- [ ] `YYYYMMDD-summons-XXX.csv`
- type: checkboxes
id: data-checklist
attributes:
label: Checklist
options:
- label: CSV files use the correct naming format (`YYYYMMDD-{type}-XXX.csv`)
- label: CSV files are in the correct location (`db/seed/updates/`)
- label: All required fields are filled out
- label: Dates use the correct format (`YYYY-MM-DD`)
- label: Boolean values are either `true` or `false`
- label: Arrays use the correct format (e.g., `{value1,value2}`)
- label: Ran import in test mode (`bin/rails granblue:import_data TEST=true`)
- type: textarea
id: test-results
attributes:
label: Test Results
description: Paste the output from running the import in test mode
validations:
required: true

View file

@ -1,26 +1,21 @@
name: Default Pull Request # Pull Request
description: For general changes and updates
body:
- type: markdown ## Description
attributes:
value: | Please provide a clear description of your changes.
## Description
Please provide a clear description of your changes. ## Changes Made
- type: textarea
id: changes <!-- List the main changes in this PR -->
attributes: <!-- Example:
label: Changes Made - Added feature X
description: List the main changes in this PR - Fixed bug Y
placeholder: "- Added feature X\n- Fixed bug Y\n- Updated documentation for Z" - Updated documentation for Z
validations: -->
required: true
- type: checkboxes ## Checklist
id: checks
attributes: - [ ] I have tested my changes
label: Checklist - [ ] I have updated relevant documentation
options: - [ ] My changes generate no new warnings
- label: I have tested my changes - [ ] I have added tests if applicable
- label: I have updated relevant documentation
- label: My changes generate no new warnings
- label: I have added tests if applicable