Skip to content
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

fix: Refine install-dev.sh and the ./py wrapper #438

Merged
merged 4 commits into from Jun 9, 2022
Merged

Conversation

fregataa
Copy link
Contributor

@fregataa fregataa commented Jun 4, 2022

modify install and pants executing scripts for detail handling such as exporting PY env and check whether a temporary env exists

@fregataa fregataa self-assigned this Jun 4, 2022
@fregataa fregataa marked this pull request as draft June 4, 2022 16:58
@achimnol achimnol added area:infrastructure Infrastructure-related issues comp:installer Related to Installer labels Jun 9, 2022
@achimnol achimnol added this to the 22.03 milestone Jun 9, 2022
@fregataa fregataa marked this pull request as ready for review June 9, 2022 09:10
@fregataa fregataa requested a review from achimnol June 9, 2022 09:10
@fregataa fregataa changed the title modify script fix: modify install and pants executing script Jun 9, 2022
@achimnol achimnol changed the title fix: modify install and pants executing script fix: Refine install-dev.sh and the pants wrapper Jun 9, 2022
@achimnol achimnol changed the title fix: Refine install-dev.sh and the pants wrapper fix: Refine install-dev.sh and the ./py wrapper Jun 9, 2022
@achimnol achimnol merged commit 38db104 into main Jun 9, 2022
@achimnol achimnol deleted the minor-script-mod branch June 9, 2022 16:05
achimnol pushed a commit that referenced this pull request Jun 9, 2022
Backported-From: main (22.06)
Backported-To: 22.03
@Yaminyam Yaminyam added the size:S 10~30 LoC label Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:infrastructure Infrastructure-related issues comp:installer Related to Installer size:S 10~30 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants