Update PR templates
This commit is contained in:
parent
72d859c4dd
commit
360391435f
4 changed files with 103 additions and 41 deletions
9
.github/PULL_REQUEST_TEMPLATE.md
vendored
Normal file
9
.github/PULL_REQUEST_TEMPLATE.md
vendored
Normal file
|
|
@ -0,0 +1,9 @@
|
|||
blank_issues_enabled: false
|
||||
contact_links:
|
||||
|
||||
- name: Default Pull Request
|
||||
url: .github/PULL_REQUEST_TEMPLATE/default.md
|
||||
about: For general changes and updates
|
||||
- name: Data Update Pull Request
|
||||
url: .github/PULL_REQUEST_TEMPLATE/data_update.md
|
||||
about: For game data updates and modifications
|
||||
68
.github/PULL_REQUEST_TEMPLATE/data_update.md
vendored
Normal file
68
.github/PULL_REQUEST_TEMPLATE/data_update.md
vendored
Normal file
|
|
@ -0,0 +1,68 @@
|
|||
name: Data Update
|
||||
description: For game data updates and modifications
|
||||
body:
|
||||
|
||||
- type: markdown
|
||||
attributes:
|
||||
value: |
|
||||
## Data Update
|
||||
- type: textarea
|
||||
id: summary
|
||||
attributes:
|
||||
label: Summary
|
||||
description: Describe what this data update includes
|
||||
placeholder: "Adding new Valentines 2024 characters"
|
||||
validations:
|
||||
required: true
|
||||
- type: textarea
|
||||
id: new-additions
|
||||
attributes:
|
||||
label: New Additions
|
||||
description: List new items being added, grouped by type
|
||||
value: |
|
||||
##### Characters
|
||||
-
|
||||
##### Weapons
|
||||
-
|
||||
##### Summons
|
||||
-
|
||||
- type: textarea
|
||||
id: modifications
|
||||
attributes:
|
||||
label: Modifications
|
||||
description: List existing items being modified
|
||||
value: |
|
||||
##### Characters
|
||||
-
|
||||
##### Weapons
|
||||
-
|
||||
##### Summons
|
||||
-
|
||||
- type: textarea
|
||||
id: csv-files
|
||||
attributes:
|
||||
label: CSV Files Added
|
||||
description: List all CSV files included in this update
|
||||
value: |
|
||||
- [ ] `YYYYMMDD-characters-XXX.csv`
|
||||
- [ ] `YYYYMMDD-weapons-XXX.csv`
|
||||
- [ ] `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
|
||||
26
.github/PULL_REQUEST_TEMPLATE/default.md
vendored
Normal file
26
.github/PULL_REQUEST_TEMPLATE/default.md
vendored
Normal file
|
|
@ -0,0 +1,26 @@
|
|||
name: Default Pull Request
|
||||
description: For general changes and updates
|
||||
body:
|
||||
|
||||
- type: markdown
|
||||
attributes:
|
||||
value: |
|
||||
## Description
|
||||
Please provide a clear description of your changes.
|
||||
- type: textarea
|
||||
id: changes
|
||||
attributes:
|
||||
label: Changes Made
|
||||
description: List the main changes in this PR
|
||||
placeholder: "- Added feature X\n- Fixed bug Y\n- Updated documentation for Z"
|
||||
validations:
|
||||
required: true
|
||||
- type: checkboxes
|
||||
id: checks
|
||||
attributes:
|
||||
label: Checklist
|
||||
options:
|
||||
- label: I have tested my changes
|
||||
- label: I have updated relevant documentation
|
||||
- label: My changes generate no new warnings
|
||||
- label: I have added tests if applicable
|
||||
41
.github/pull_request_template.md
vendored
41
.github/pull_request_template.md
vendored
|
|
@ -1,41 +0,0 @@
|
|||
## Data Update
|
||||
|
||||
### Summary
|
||||
<!-- Describe what this data update includes. For example: "Adding new Valentines 2024 characters" -->
|
||||
|
||||
#### New Additions
|
||||
<!-- List new items being added, grouped by type -->
|
||||
- Characters:
|
||||
- [ ]
|
||||
- Weapons:
|
||||
- [ ]
|
||||
- Summons:
|
||||
- [ ]
|
||||
|
||||
#### Modifications
|
||||
<!-- List existing items being modified -->
|
||||
- Characters:
|
||||
- [ ]
|
||||
- Weapons:
|
||||
- [ ]
|
||||
- Summons:
|
||||
- [ ]
|
||||
|
||||
### CSV Files Added
|
||||
<!-- List all CSV files included in this update -->
|
||||
- [ ] `YYYYMMDD-characters-XXX.csv`
|
||||
- [ ] `YYYYMMDD-weapons-XXX.csv`
|
||||
- [ ] `YYYYMMDD-summons-XXX.csv`
|
||||
|
||||
### Checklist
|
||||
<!-- Make sure all these items are completed -->
|
||||
- [ ] CSV files use the correct naming format (`YYYYMMDD-{type}-XXX.csv`)
|
||||
- [ ] CSV files are in the correct location (`db/seed/updates/`)
|
||||
- [ ] All required fields are filled out
|
||||
- [ ] Dates use the correct format (`YYYY-MM-DD`)
|
||||
- [ ] Boolean values are either `true` or `false`
|
||||
- [ ] Arrays use the correct format (e.g., `{value1,value2}`)
|
||||
- [ ] Ran import in test mode (`bin/rails granblue:import_data TEST=true`)
|
||||
|
||||
### Test Results
|
||||
<!-- Paste the output from running the import in test mode -->
|
||||
Loading…
Reference in a new issue