Skip to content

protobuf should use PYEXE not pip#5231

Merged
wyli merged 1 commit intoProject-MONAI:devfrom
rijobro:protobuf_pip
Sep 29, 2022
Merged

protobuf should use PYEXE not pip#5231
wyli merged 1 commit intoProject-MONAI:devfrom
rijobro:protobuf_pip

Conversation

@rijobro
Copy link
Copy Markdown
Contributor

@rijobro rijobro commented Sep 29, 2022

Description

We should be using PY_EXE in runtests as opposed to raw pip.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).

@rijobro rijobro requested a review from wyli September 29, 2022 16:28
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Sep 29, 2022

/build

@wyli wyli enabled auto-merge (squash) September 29, 2022 16:50
@wyli wyli merged commit 71041fd into Project-MONAI:dev Sep 29, 2022
wyli pushed a commit that referenced this pull request Oct 10, 2022
### Description

We should be using `PY_EXE` in runtests as opposed to raw `pip`.

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).

Signed-off-by: Richard Brown <[email protected]>
KumoLiu pushed a commit that referenced this pull request Nov 2, 2022
### Description

We should be using `PY_EXE` in runtests as opposed to raw `pip`.

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).

Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: KumoLiu <[email protected]>
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