-
Notifications
You must be signed in to change notification settings - Fork 277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prepend to PYTHONPATH in tests #2673
Conversation
Preserve the existing PYTHONPATH by prepending instead of replacing. Signed-off-by: Steve Peters <[email protected]>
Signed-off-by: Steve Peters <[email protected]>
I think both e527979 and gazebo-tooling/release-tools#1203 are needed to fix the python system loader integration test |
I'm going to temporarily reconfigure the homebrew jenkins job to use the release-tools branch from gazebo-tooling/release-tools#1203 |
@osrf-jenkins run tests please |
https://build.osrfoundation.org/job/gz_sim-ci-pr_any-homebrew-amd64/1074/ |
https://github.com/Mergifyio backport gz-sim8 |
✅ Backports have been created
|
Preserve the existing PYTHONPATH by prepending instead of replacing. Signed-off-by: Steve Peters <[email protected]> (cherry picked from commit 0d346de)
Preserve the existing PYTHONPATH by prepending instead of replacing. Signed-off-by: Steve Peters <[email protected]> (cherry picked from commit 0d346de) Co-authored-by: Steve Peters <[email protected]>
🦟 Bug fix
Fixes failing python tests
Summary
Preserve the existing PYTHONPATH by prepending instead of replacing. This is needed since we started using a virtual environment in CI.
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.