From 17062a689604f9e3db17eb938dc82890844d3ec5 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 30 Oct 2025 11:08:00 +0000 Subject: [PATCH] Simplify runnerSize logic and add clarifying comments Co-authored-by: henrymercer <14129055+henrymercer@users.noreply.github.com> --- pr-checks/sync.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/pr-checks/sync.py b/pr-checks/sync.py index d0725eaee..a8ccf817a 100755 --- a/pr-checks/sync.py +++ b/pr-checks/sync.py @@ -76,17 +76,19 @@ for file in sorted((this_dir / 'checks').glob('*.yml')): if version == "latest": raise ValueError('Did not recognize "version: latest". Did you mean "version: linked"?') - # Determine runner size (default is "latest") - # "default" maps to "latest" for the runner image name + # Determine runner size + # "default" is used in check specifications and maps to "latest" for the actual runner image runnerSize = checkSpecification.get('runnerSize', 'default') - actualRunnerSize = 'latest' if runnerSize == 'default' else runnerSize + if runnerSize == 'default': + runnerSize = 'latest' # Build runner images based on runner size and operating systems operatingSystems = checkSpecification.get('operatingSystems', ["ubuntu"]) for operatingSystem in operatingSystems: # Construct the runner image name: {os}-{size} - runnerImage = f"{operatingSystem}-{actualRunnerSize}" + # Note: Not all OS types may support all runner sizes (e.g., only ubuntu-slim exists as of now) + runnerImage = f"{operatingSystem}-{runnerSize}" matrix.append({ 'os': runnerImage, 'version': version