Skip to content

[LatencyTest] Fixing more path issues in platform_scripts (#604)#606

Merged
mcottontensor merged 1 commit intoEpicGamesExt:LatencyTestfrom
mcottontensor:backport/LatencyTest/pr-604
May 4, 2025
Merged

[LatencyTest] Fixing more path issues in platform_scripts (#604)#606
mcottontensor merged 1 commit intoEpicGamesExt:LatencyTestfrom
mcottontensor:backport/LatencyTest/pr-604

Conversation

@mcottontensor
Copy link
Copy Markdown
Collaborator

Backport

This will backport the following commits from master to LatencyTest:

Questions ?

Please refer to the Backport tool documentation

* feat: Adding script tests where the scripts are run multiple times.

This tests that the scripts still work when things have already been
installed.

* fix: Fixing typo in github action.

* gimme a break.

* Adding mutliple script runs for other OSs

* Testing node on the runner

* Trying to figure out what node is doing what.

* Trying to change the way node version is detected.

* Messy

* The additions to the script really didn't catch anything. Could be worth
revisiting how we handle the local node install.

* Removing debug print.

(cherry picked from commit 2321fa9)
@mcottontensor mcottontensor merged commit 07159b1 into EpicGamesExt:LatencyTest May 4, 2025
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented May 4, 2025

⚠️ No Changeset found

Latest commit: d86e50c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mcottontensor mcottontensor deleted the backport/LatencyTest/pr-604 branch May 4, 2025 23:28
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.

1 participant