Skip to content

Conversation

@williammorrison2
Copy link
Owner

Purpose

Describe the problem or feature. Ask yourself, "What is the goal of this pull request?"

Approach

How did you address the problem? What was your thinking process?

Origin

Link the origin git issue here. and describe any offline discussion if applicable.

How to Test

This section will help the regression tester.

  1. Think about what you want to test.
  2. Test it.

Screenshots

Place appropriate screenshots or code snippets here.

Open Questions and Pre-Merge TODOs

Make sure you attempted to do the following:

  • Use github checklists. When solved, check the box and explain the answer.
  • Assign both @pdlun92 and @williammorrison2 to review (So they can check regression)
  • Assign 1 developer to sanity check the code. (Working on automatically doing this via ./github/CODEOWNERS.md)
  • Replace suggested text in this template with real PR information.
  • Appropriatly label
  • Assign a milestone.
  • update appropriate documentation in the /docs folder for your code changes.
  • Remove Open Questions and Pre-Merge TODOs when you no longer need it.

@williammorrison2 williammorrison2 self-assigned this Aug 13, 2019
@williammorrison2 williammorrison2 added the enhancement New feature or request label Aug 13, 2019
@williammorrison2 williammorrison2 added the csm_fvt_regression This label kicks off the automated regression process label Aug 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

csm_fvt_regression This label kicks off the automated regression process enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants