Skip to content

Conversation

@manopapad
Copy link
Contributor

No description provided.

@manopapad manopapad requested a review from bryevdv August 11, 2022 01:16
@bryevdv
Copy link
Contributor

bryevdv commented Aug 11, 2022

I guess I thought they were supposed to combine. LGTM but there is probably a test here that needs to be updated:

https://github.com/nv-legate/cunumeric/blob/branch-22.10/tests/_utils/tests/test_config.py

Those test files are runnable with standard pytest

@manopapad manopapad merged commit b4fbde3 into nv-legate:branch-22.10 Aug 15, 2022
sbak5 pushed a commit to sbak5/cunumeric that referenced this pull request Aug 17, 2022
* Ensure test.py --use flag fully overrides USE_* envvars

* Update a test-tools unit test

Co-authored-by: Manolis Papadakis <mpapadakis@nvidia.com>
@manopapad manopapad deleted the arg-overrides-envvar branch July 19, 2023 17:29
manopapad pushed a commit that referenced this pull request Feb 18, 2025
* enhance for test_squeeze.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants