Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[No QA] Add QA steps to the pull request template #2091

Merged
merged 1 commit into from
Mar 26, 2021

Conversation

roryabraham
Copy link
Contributor

Details

Updates the Expensify.cash pull request template to include a section with QA steps for staging QA.

Fixed Issues

n/a – Talked with @isagoico 1:1 about how many PRs do not have clear QA steps that work on staging. Providing QA steps is a standard practice in all our other repos, so it should be a standard practice in this one too.

Tests

  1. Merge this pull request.
  2. Create a new pull request.
  3. Ensure that the new pull request is pre-populated with a section titled "QA Steps"

Tested On

n/a - can only be tested on Github by merging.

@roryabraham roryabraham requested a review from a team March 25, 2021 22:09
@roryabraham roryabraham self-assigned this Mar 25, 2021
@botify botify requested review from sketchydroide and removed request for a team March 25, 2021 22:09
Copy link
Contributor

@sketchydroide sketchydroide left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sketchydroide sketchydroide merged commit fa6dbb2 into master Mar 26, 2021
@sketchydroide sketchydroide deleted the Rory-UpdatePRTemplate branch March 26, 2021 15:48
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@sketchydroide
Copy link
Contributor

Did the test as well and it passes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants