Skip to content

[No QA]Sign cherry pick PRs as @OSBotify#6655

Merged
roryabraham merged 1 commit into
mainfrom
andrew-sign-cp
Dec 9, 2021
Merged

[No QA]Sign cherry pick PRs as @OSBotify#6655
roryabraham merged 1 commit into
mainfrom
andrew-sign-cp

Conversation

@AndrewGable

@AndrewGable AndrewGable commented Dec 8, 2021

Copy link
Copy Markdown
Contributor

Details

This PR should sign our cherry pick commits and make all the cherry picks created by @OSBotify. This is the last step to forcing signatures for all commits.

Fixed Issues

$ GH_LINK

Tests

  1. Merge this PR with the CP label and the QA checklist locked
  2. Make sure the commits to the staging branch are signed and verified
  3. Once 2 is confirmed, create a test CP with a dummy change to CP manually

@AndrewGable AndrewGable self-assigned this Dec 8, 2021
@AndrewGable AndrewGable requested a review from a team as a code owner December 8, 2021 23:33
@MelvinBot MelvinBot requested review from danieldoglas and removed request for a team December 8, 2021 23:33

@Jag96 Jag96 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, can we also add manual cherry-picking to the test list just to confirm that all still works?


- name: Create branch for new pull request
run: |
git config user.name ${{ github.actor }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this will fail if we don't specify a user name. I think we need to hardcode it to OSBotify if we want all CP commits to be done + signed by OSBotify:

git config user.name 'OSBotify'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@github-actions

github-actions Bot commented Dec 9, 2021

Copy link
Copy Markdown
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@roryabraham

Copy link
Copy Markdown
Contributor

Let's see if this works

@roryabraham roryabraham merged commit 7068258 into main Dec 9, 2021
@roryabraham roryabraham deleted the andrew-sign-cp branch December 9, 2021 22:23
OSBotify pushed a commit that referenced this pull request Dec 9, 2021
@roryabraham

Copy link
Copy Markdown
Contributor

Seems like it worked for a label CP:

image

@roryabraham

Copy link
Copy Markdown
Contributor

Testing this with a manual CP of #6684

@roryabraham

Copy link
Copy Markdown
Contributor

This seemed to work with a manual CP as well 🎉

image

@AndrewGable

Copy link
Copy Markdown
Contributor Author

Awesome!! Thank you for testing this!

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @roryabraham in version: 1.1.19-5 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @Julesssss in version: 1.1.21-1 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

4 participants