Skip to content

Conversation

sam-github
Copy link
Contributor

Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

Backport-of: #1844

Checklist
  • npm install && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

Backport-of: #1844
@joaocgreis
Copy link
Member

#1909 includes EXPERIMENTAL_NODE_GYP_PYTHON3. The second PR to land should remove it from there.

rvagg pushed a commit that referenced this pull request Oct 4, 2019
Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

rvagg: removed additional EXPERIMENTAL_NODE_GYP_PYTHON3 in
.travis.yml from a previous commit while landing.

Backport-of: #1844
PR-URL: #1910
Reviewed-By: Christian Clauss <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: João Reis <[email protected]>
@rvagg
Copy link
Member

rvagg commented Oct 4, 2019

sweet, landed in v5.x with additional EXPERIMENTAL_NODE_GYP_PYTHON3 removed

@rvagg rvagg closed this Oct 4, 2019
@rvagg rvagg mentioned this pull request Oct 4, 2019
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg added a commit that referenced this pull request Oct 4, 2019
Dropped accidentally in backport @
#1910

The change to >=2.7 was declared semver-major in
#1813
rvagg pushed a commit that referenced this pull request Oct 4, 2019
Python 3 is allowed as a compatible Python, but its looked for after
Python 2.

rvagg: removed additional EXPERIMENTAL_NODE_GYP_PYTHON3 in
.travis.yml from a previous commit while landing.

Backport-of: #1844
PR-URL: #1910
Reviewed-By: Christian Clauss <[email protected]>
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: João Reis <[email protected]>
@sam-github sam-github deleted the allow-py3-after-py2 branch October 25, 2019 14:43
@cclauss cclauss added the Python label Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants