Skip to content

Move run-test-repeatedly scripts to .github/workflows/fix-flaky-test/#14704

Merged
radical merged 4 commits into
mainfrom
copilot/move-scripts-to-fix-flaky-test
Feb 26, 2026
Merged

Move run-test-repeatedly scripts to .github/workflows/fix-flaky-test/#14704
radical merged 4 commits into
mainfrom
copilot/move-scripts-to-fix-flaky-test

Conversation

Copilot AI commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Relocates the run-test-repeatedly.sh and run-test-repeatedly.ps1 scripts from the repo root into .github/workflows/fix-flaky-test/ to co-locate them with the related flaky test tooling.

Description

Changes:

  • Moved run-test-repeatedly.{sh,ps1}.github/workflows/fix-flaky-test/
  • Updated all path references in .github/skills/fix-flaky-test/SKILL.md to use the ./.github/workflows/fix-flaky-test/run-test-repeatedly.* prefix for copy-paste-friendly invocation from the repo root
  • Updated self-referencing usage/help text inside both scripts

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
      • If yes, did you have an API Review for it?
        • Yes
        • No
      • Did you add <remarks /> and <code /> elements on your triple slash comments?
        • Yes
        • No
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
      • If yes, have you done a threat model and had a security review?
        • Yes
        • No
    • No
  • Does the change require an update in our Aspire docs?

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: radical <1472+radical@users.noreply.github.com>
Copilot AI changed the title [WIP] Move run-test-repeatedly scripts to .github/workflows/fix-flaky-test directory Move run-test-repeatedly scripts to .github/workflows/fix-flaky-test/ Feb 26, 2026
@github-actions

github-actions Bot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 14704

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 14704"

@radical radical marked this pull request as ready for review February 26, 2026 00:40
@radical radical added the area-engineering-systems infrastructure helix infra engineering repo stuff label Feb 26, 2026

Copilot AI 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.

Pull request overview

Relocates the run-test-repeatedly helper scripts into .github/workflows/fix-flaky-test/ and updates documentation/help text so the flaky-test tooling is co-located and usage guidance matches the new location.

Changes:

  • Moved run-test-repeatedly.sh and run-test-repeatedly.ps1 under .github/workflows/fix-flaky-test/.
  • Updated script usage/help output to reference the new script paths.
  • Updated .github/skills/fix-flaky-test/SKILL.md to reference the new script location and invocation examples.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
.github/workflows/fix-flaky-test/run-test-repeatedly.sh Updates usage/examples to reflect the new script location.
.github/workflows/fix-flaky-test/run-test-repeatedly.ps1 Updates usage/examples to reflect the new script location.
.github/skills/fix-flaky-test/SKILL.md Updates documentation and command snippets to point to the relocated scripts.
Comments suppressed due to low confidence (3)

.github/workflows/fix-flaky-test/run-test-repeatedly.sh:15

  • Consider including an explicit current-directory prefix in the printed usage (e.g., ./.github/workflows/fix-flaky-test/run-test-repeatedly.sh ...) to make copy/paste invocation clearer and consistent with other repo docs/scripts.
    .github/workflows/fix-flaky-test/run-test-repeatedly.ps1:21
  • PowerShell script invocation examples here use .github/workflows/.../run-test-repeatedly.ps1 without .//.\/& prefix. In PowerShell, relative script execution typically requires an explicit relative path (e.g., ./.github/workflows/fix-flaky-test/run-test-repeatedly.ps1 ...) or pwsh -File ...; otherwise copy/paste often fails due to command resolution rules.
    .github/workflows/fix-flaky-test/run-test-repeatedly.ps1:49
  • Same as the comment above: the help text prints Usage: .github/.../run-test-repeatedly.ps1 ... without an explicit relative-path prefix. Consider printing ./.github/workflows/fix-flaky-test/run-test-repeatedly.ps1 ... (or suggesting pwsh -File ...) so the usage works when pasted into a PowerShell prompt.

Comment thread .github/skills/fix-flaky-test/SKILL.md Outdated
Comment thread .github/skills/fix-flaky-test/SKILL.md Outdated
Comment thread .github/skills/fix-flaky-test/SKILL.md Outdated

# Linux/macOS
./run-test-repeatedly.sh -n 20 -- \
.github/workflows/fix-flaky-test/run-test-repeatedly.sh -n 20 -- \

Copilot AI Feb 26, 2026

Copy link

Choose a reason for hiding this comment

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

Same issue as above: consider using ./.github/workflows/fix-flaky-test/run-test-repeatedly.sh ... instead of .github/... for clearer bash invocation from the repo root.

Copilot uses AI. Check for mistakes.
Comment thread .github/skills/fix-flaky-test/SKILL.md Outdated
Comment thread .github/skills/fix-flaky-test/SKILL.md Outdated
Comment thread .github/skills/fix-flaky-test/SKILL.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: radical <1472+radical@users.noreply.github.com>
@radical radical enabled auto-merge (squash) February 26, 2026 00:52
@radical radical merged commit d22b484 into main Feb 26, 2026
671 of 675 checks passed
@radical radical deleted the copilot/move-scripts-to-fix-flaky-test branch February 26, 2026 01:34
@dotnet-policy-service dotnet-policy-service Bot added this to the 13.3 milestone Feb 26, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-engineering-systems infrastructure helix infra engineering repo stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants