Skip to content

Port to python3#27

Merged
MaxSem merged 1 commit intomasterfrom
python3
Jul 17, 2019
Merged

Port to python3#27
MaxSem merged 1 commit intomasterfrom
python3

Conversation

@MaxSem
Copy link
Contributor

@MaxSem MaxSem commented Jun 28, 2019

Bug: T219472

Bug: T219472
Copy link

@aezell aezell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good with just the one change to the venv typo.

Virtualenv is recommended:

virtualenv venv
python3 -m venv venv
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think venv is duplicated here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The first one is module name, the second one is destination directory.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had no idea. Thanks!

@MaxSem MaxSem merged commit d623ba5 into master Jul 17, 2019
@MaxSem MaxSem deleted the python3 branch July 17, 2019 02:43
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