Skip to content

[No QA] Use actions dir for setupNode action#9862

Merged
roryabraham merged 2 commits into
mainfrom
Rory-FixActionPath
Jul 12, 2022
Merged

[No QA] Use actions dir for setupNode action#9862
roryabraham merged 2 commits into
mainfrom
Rory-FixActionPath

Conversation

@roryabraham

@roryabraham roryabraham commented Jul 12, 2022

Copy link
Copy Markdown
Contributor

Details

Following-up on #9815 to fix some paths.

Fixed Issues

$ n/a

Tests

  1. Merge this PR
  2. Tests and lint will run and should pass

@roryabraham roryabraham requested a review from a team as a code owner July 12, 2022 21:22
@roryabraham roryabraham self-assigned this Jul 12, 2022
@AndrewGable AndrewGable requested review from AndrewGable and removed request for a team July 12, 2022 21:22
@melvin-bot melvin-bot Bot requested a review from jasperhuangg July 12, 2022 21:22
AndrewGable
AndrewGable previously approved these changes Jul 12, 2022
@roryabraham roryabraham merged commit 90c4629 into main Jul 12, 2022
@roryabraham roryabraham deleted the Rory-FixActionPath branch July 12, 2022 21:43
@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.

@AndrewGable

Copy link
Copy Markdown
Contributor

Tests will fail until we merge because they use the code on main. Merging to fix tests!

@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