Skip to content

Minor amendment to Python tests - delete venv, uv.lock symlinks after tests#6872

Merged
greg-at-moderne merged 1 commit intomainfrom
greg-fix-symlink-temp-warnings
Mar 5, 2026
Merged

Minor amendment to Python tests - delete venv, uv.lock symlinks after tests#6872
greg-at-moderne merged 1 commit intomainfrom
greg-fix-symlink-temp-warnings

Conversation

@greg-at-moderne
Copy link
Contributor

@greg-at-moderne greg-at-moderne commented Mar 4, 2026

What's changed?

Removing venv, uv.lock symlinks after some of the Python tests.

What's your motivation?

To save us from warnings being printed out when executing them, e.g.

WARNING: Deleting symbolic link from location inside of temp dir (/tmp/junit-4993433042930029754/uv.lock) to location outside of temp dir (/tmp/openrewrite-python-workspaces/QbrGqW7Ph2tyPyzs/uv.lock) but not the target file/directory
DependencyInsightTest > findTransitiveDependencyWithDataTable(Path) STANDARD_ERROR

@greg-at-moderne greg-at-moderne self-assigned this Mar 4, 2026
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Mar 4, 2026
@greg-at-moderne greg-at-moderne marked this pull request as ready for review March 5, 2026 05:31
@greg-at-moderne greg-at-moderne merged commit d5286e7 into main Mar 5, 2026
1 check passed
@greg-at-moderne greg-at-moderne deleted the greg-fix-symlink-temp-warnings branch March 5, 2026 05:31
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant