Skip to content

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Aug 24, 2019

bytes and str changes to inspector_protocol for Python 3 compatibility.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Aug 24, 2019
@cclauss cclauss requested review from Trott, bnoordhuis and targos August 24, 2019 19:12
@cclauss cclauss added the python PRs and issues that require attention from people who are familiar with Python. label Aug 24, 2019
@cclauss cclauss requested a review from trivikr August 24, 2019 19:20
@nodejs-github-bot
Copy link
Collaborator

@Trott
Copy link
Member

Trott commented Aug 27, 2019

Landed in 4662f67

@Trott Trott closed this Aug 27, 2019
Trott pushed a commit that referenced this pull request Aug 27, 2019
PR-URL: #29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@cclauss cclauss deleted the py3_fixes_in_inspector_protocol branch August 27, 2019 05:34
BridgeAR pushed a commit that referenced this pull request Sep 3, 2019
PR-URL: #29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@BridgeAR BridgeAR mentioned this pull request Sep 3, 2019
BridgeAR pushed a commit that referenced this pull request Sep 4, 2019
PR-URL: #29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
BaochengSu pushed a commit to BaochengSu/node that referenced this pull request Oct 20, 2020
PR-URL: nodejs#29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
(cherry picked from commit 4662f67)
aduh95 pushed a commit to aduh95/node that referenced this pull request Mar 2, 2021
PR-URL: nodejs#29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
aduh95 pushed a commit to aduh95/node that referenced this pull request Mar 3, 2021
PR-URL: nodejs#29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
aduh95 pushed a commit to aduh95/node that referenced this pull request Mar 4, 2021
PR-URL: nodejs#29296
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python PRs and issues that require attention from people who are familiar with Python. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants