Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

docs: update pr template #727

Merged
merged 1 commit into from
Nov 19, 2022
Merged

docs: update pr template #727

merged 1 commit into from
Nov 19, 2022

Conversation

LordOfPolls
Copy link
Member

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change
  • CI change
  • Other: GitHub Template Update

Description

Increases verbosity of GitHub template, corrects grammar, and adds needed sections

Changes

  • Bullet pointed -> bullet-pointed
  • Add Test Scenario(s) section
  • Add additional entries to checklist
  • Add additional entries to PR type

Test Scenario(s)

non-code change

Checklist

  • I've formatted my code with Black
  • I've added docstrings to everything I've touched
  • I've ensured my code works on Python 3.10.x
  • I've ensured my code works on Python 3.11.x
  • I've tested my changes
  • I've added tests for my code - if applicable
  • I've updated the documentation - if applicable

@LordOfPolls LordOfPolls added the Documentation A documentation change label Nov 14, 2022
@LordOfPolls LordOfPolls merged commit 13c592e into dev Nov 19, 2022
@LordOfPolls LordOfPolls deleted the pr_template branch November 19, 2022 13:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
Documentation A documentation change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@LordOfPolls