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] Ensure automerge PRs are not included in StagingDeployCash #2020

Merged
merged 3 commits into from
Mar 31, 2021

Conversation

roryabraham
Copy link
Contributor

Details

Right now, the StagingDeployCash issue includes version-bump PRs that are auto-generated and auto-merged. These do not need to be included in our QA checklists.

Fixed Issues

Fixes https://github.com/Expensify/Expensify/issues/157810

Tests

Added automated tests. To live-test:

  1. Merge this PR
  2. Ensure that the StagingDeployCash is updated as part of the preDeploy Github workflow.
  3. After being updated, the StagingDeployCash should not include any version-bump PRs.

@roryabraham roryabraham requested a review from AndrewGable March 23, 2021 21:21
@roryabraham roryabraham self-assigned this Mar 23, 2021
@roryabraham roryabraham requested a review from a team as a code owner March 23, 2021 21:21
@botify botify requested review from TomatoToaster and removed request for a team March 23, 2021 21:21
@roryabraham roryabraham changed the title Ensure automerge PRs are not included in StagingDeployCash [HOLD PR #1947] Ensure automerge PRs are not included in StagingDeployCash Mar 23, 2021
@roryabraham roryabraham changed the title [HOLD PR #1947] Ensure automerge PRs are not included in StagingDeployCash Ensure automerge PRs are not included in StagingDeployCash Mar 25, 2021
@roryabraham
Copy link
Contributor Author

Okay, no longer on hold. Ready for review

@TomatoToaster
Copy link
Contributor

Hey @roryabraham, since the testing steps mentioned that you'd start with merging this PR, I figured it'd make sense to wait for you to merge it and test it.

Let me know if there's any way I can help with the testing.

@roryabraham
Copy link
Contributor Author

Ah yes. Thanks @TomatoToaster, I can test this! Need to wait until the deploy checklist doesn't have the lock label tho 🙃

@roryabraham roryabraham merged commit e9f6d58 into master Mar 31, 2021
@roryabraham roryabraham deleted the Rory-NoVersionBumpInStagingDeploy branch March 31, 2021 16:03
@roryabraham roryabraham changed the title Ensure automerge PRs are not included in StagingDeployCash [No QA] Ensure automerge PRs are not included in StagingDeployCash Mar 31, 2021
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