Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

mike-weiner
Copy link
Contributor

@mike-weiner mike-weiner commented Sep 3, 2025

Description

Closes #15200.

This PR refactors scripts/pr_tests.py to instead use the files changed portion of a PR to determine which snips require testing.

Proof of Testing

@craigbox left some examples in #15200 of where problems were encountered. I used those as my testing for the updates to this script.

For folks that want to test, installing uv is straightforward and makes the dependency management of Python much easier. Installation instructions are here.

Correctly found all snips requiring testing in a PR:

weiner istio.io (master) % uv run --with PyGithub ./scripts/pr_tests.py --token= 15168
ambient/getting-started,ambient/getting-started/cleanup,ambient/getting-started/deploy-sample-app,ambient/getting-started/enforce-auth-policies,ambient/getting-started/manage-traffic,ambient/getting-started/secure-and-visualize,ambient/getting-started/setup,ambient/usage/extend-waypoint-wasm

Correctly did not test snips being removed in a PR:

weiner istio.io (master) % uv run --with PyGithub ./scripts/pr_tests.py --token= 15227
NONE

Reviewers

  • Ambient
  • Docs
  • Installation
  • Networking
  • Performance and Scalability
  • Extensions and Telemetry
  • Security
  • Test and Release
  • User Experience
  • Developer Infrastructure
  • Localization/Translation

@mike-weiner mike-weiner requested a review from a team as a code owner September 3, 2025 00:56
@istio-testing istio-testing added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/docs size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test scripts break when a folder with snips is created and removed in the same PR
3 participants