Skip to content

[No QA] Change job outputs to UPPER_SNAKE_CASE#9866

Merged
roryabraham merged 1 commit into
mainfrom
Rory-FixPreDeploy
Jul 12, 2022
Merged

[No QA] Change job outputs to UPPER_SNAKE_CASE#9866
roryabraham merged 1 commit into
mainfrom
Rory-FixPreDeploy

Conversation

@roryabraham

Copy link
Copy Markdown
Contributor

Details

We are unsure why the createNewVersion job is being skipped, and we're unable to reproduce the problem in the test repo.

So now I'm just wanting to retry to see if it's some GitHub bug / fluke, and this PR puts this workflow inline with our existing conventions for job outputs.

Fixed Issues

$ n/a

Tests

  1. Merge this PR
  2. Hope it triggers a deploy.

@roryabraham roryabraham requested a review from a team as a code owner July 12, 2022 23:06
@roryabraham roryabraham self-assigned this Jul 12, 2022
@melvin-bot melvin-bot Bot requested review from alex-mechler and removed request for a team July 12, 2022 23:06
@AndrewGable AndrewGable requested review from AndrewGable and removed request for alex-mechler July 12, 2022 23:11
@roryabraham roryabraham merged commit 3130140 into main Jul 12, 2022
@roryabraham roryabraham deleted the Rory-FixPreDeploy branch July 12, 2022 23:12
@melvin-bot melvin-bot Bot added the Emergency label Jul 12, 2022
@melvin-bot

melvin-bot Bot commented Jul 12, 2022

Copy link
Copy Markdown

@roryabraham looks like this was merged without passing tests. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@roryabraham

Copy link
Copy Markdown
Contributor Author

Tests were passing...

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @chiragsalian in version: 1.1.84-13 🚀

platform result
🤖 android 🤖 success ✅
🖥 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.

3 participants