Skip to content

[No QA] Ensure automerge PRs are not included in StagingDeployCash#2020

Merged
roryabraham merged 3 commits into
masterfrom
Rory-NoVersionBumpInStagingDeploy
Mar 31, 2021
Merged

[No QA] Ensure automerge PRs are not included in StagingDeployCash#2020
roryabraham merged 3 commits into
masterfrom
Rory-NoVersionBumpInStagingDeploy

Conversation

@roryabraham

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

Okay, no longer on hold. Ready for review

@TomatoToaster

Copy link
Copy Markdown
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
Copy Markdown
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