Skip to content

Latest commit

 

History

History
40 lines (30 loc) · 1.81 KB

PULL_REQUEST_TEMPLATE.md

File metadata and controls

40 lines (30 loc) · 1.81 KB

Description

  • Please include a summary of the change and which issue is fixed.
  • List any dependencies that are required for this change.

Fixes # (issue number)

Type of change

Please delete options that are not relevant

  • [] Bug fix (non-breaking change which fixes an issue)
  • [] New Feature (non-breaking change which adds functionality)
  • [] Breaking change(fix or feature that would cause existing functionality to not work as expected)
  • [] This change requires a documentation update

Testing locally

  • Add here anything someone would need to do to test your code locally, or delete this section if nothing needed
  • E.g. run npm install in the app, refresh the database, look at a particular project that has a video, add a .env file value (NB do not share any sensitive details here as PRs are public)

Checklist

  • [] My code follows the style guidelines of this project
  • [] I have performed a self-review of my own code
  • [] I have commented my code, particularly in hard-to-understand areas
  • [] I have removed any unnecessary comments or console logging
  • [] I have made corresponding changes to the documentation (if required)
  • [] I have addressed accessibility, if needed
  • [] I have followed best practices, e.g. NativeBase approaches and theming
  • [] I have checked the app in dark mode, if making front-end design changes
  • [] My changes generate no new warnings
  • [] I have added tests that prove my fix is effective or that my feature works
  • [] New and existing unit tests pass locally with my changes
  • [] I have updated the version numbers in package.json files in the app and/or api directories as needed