| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:6f85be7d-2a2c-b653-79a3-e625832644ca |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | BF7C:BCEB:207A1D7:29F7A7E:699287E5 |
| html-safe-nonce | 3de5a5ce106bb4c3a20937f77e751fa69acbaf664f6b8ea16e43de4c3ffb6e3c |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCRjdDOkJDRUI6MjA3QTFENzoyOUY3QTdFOjY5OTI4N0U1IiwidmlzaXRvcl9pZCI6IjcyNTQ2NDU4MzE0Mjg2Mzg2OTMiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 9d7f2ec44f9a9816708c58c080e499f5d8dc67eecac4342f7524f1e9bb43cda6 |
| hovercard-subject-tag | pull_request:284361160 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/nodejs/node/28033/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/cdb3c0bad3fe419332f48e834db10a54ad795499f847d2641d94d78a2949920c/nodejs/node/pull/28033 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/cdb3c0bad3fe419332f48e834db10a54ad795499f847d2641d94d78a2949920c/nodejs/node/pull/28033 |
| og:image:alt | If the transfer list argument is present, it should be an array.
This commit adds typechecking to that effect. This aligns behaviour
with browsers.
I’m also more than happy to label this semver-maj... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | addaleax |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-cache |
| go-import | github.com/nodejs/node git https://github.com/nodejs/node.git |
| octolytics-dimension-user_id | 9950313 |
| octolytics-dimension-user_login | nodejs |
| octolytics-dimension-repository_id | 27193779 |
| octolytics-dimension-repository_nwo | nodejs/node |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 27193779 |
| octolytics-dimension-repository_network_root_nwo | nodejs/node |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://github.com/nodejs/node/pull/28033#start-of-content |
|
| https://github.com/ |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fpull%2F28033 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub SparkBuild and deploy intelligent apps | https://github.com/features/spark |
| GitHub ModelsManage and compare prompts | https://github.com/features/models |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| GitHub SponsorsFund open source developers | https://github.com/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/accelerator |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fpull%2F28033 |
|
Sign up
| https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=nodejs%2Fnode |
| Reload | https://github.com/nodejs/node/pull/28033 |
| Reload | https://github.com/nodejs/node/pull/28033 |
| Reload | https://github.com/nodejs/node/pull/28033 |
|
nodejs
| https://github.com/nodejs |
| node | https://github.com/nodejs/node |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
|
Notifications
| https://github.com/login?return_to=%2Fnodejs%2Fnode |
|
Fork
34.8k
| https://github.com/login?return_to=%2Fnodejs%2Fnode |
|
Star
116k
| https://github.com/login?return_to=%2Fnodejs%2Fnode |
|
Code
| https://github.com/nodejs/node |
|
Issues
1.7k
| https://github.com/nodejs/node/issues |
|
Pull requests
716
| https://github.com/nodejs/node/pulls |
|
Actions
| https://github.com/nodejs/node/actions |
|
Projects
3
| https://github.com/nodejs/node/projects |
|
Security
0
| https://github.com/nodejs/node/security |
|
Insights
| https://github.com/nodejs/node/pulse |
|
Code
| https://github.com/nodejs/node |
|
Issues
| https://github.com/nodejs/node/issues |
|
Pull requests
| https://github.com/nodejs/node/pulls |
|
Actions
| https://github.com/nodejs/node/actions |
|
Projects
| https://github.com/nodejs/node/projects |
|
Security
| https://github.com/nodejs/node/security |
|
Insights
| https://github.com/nodejs/node/pulse |
| addaleax | https://github.com/addaleax |
| nodejs:master | https://github.com/nodejs/node/tree/master |
| addaleax:worker-postmessage-transferlist-typecheck | https://github.com/addaleax/node/tree/worker-postmessage-transferlist-typecheck |
| Conversation | https://github.com/nodejs/node/pull/28033 |
| Commits4 (4) | https://github.com/nodejs/node/pull/28033/commits |
| Checks | https://github.com/nodejs/node/pull/28033/checks |
| Files changed | https://github.com/nodejs/node/pull/28033/files |
| worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/pull/28033#top |
| addaleax | https://github.com/addaleax |
| nodejs:master | https://github.com/nodejs/node/tree/master |
| addaleax:worker-postmessage-transferlist-typecheck | https://github.com/addaleax/node/tree/worker-postmessage-transferlist-typecheck |
|
| https://github.com/addaleax |
| addaleax | https://github.com/addaleax |
| Jun 2, 2019 | https://github.com/nodejs/node/pull/28033#issue-451219932 |
| commit guidelines | https://github.com/nodejs/node/blob/master/doc/guides/contributing/pull-requests.md#commit-message-guidelines |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
|
| https://github.com/addaleax |
| worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/pull/28033/commits/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| 7580ecb | https://github.com/nodejs/node/pull/28033/commits/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| https://github.com/addaleax |
| addaleax | https://github.com/addaleax |
|
worker
| https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Aworker |
| Jun 2, 2019 | https://github.com/nodejs/node/pull/28033#event-2383323503 |
| https://github.com/Trott |
| Trott | https://github.com/Trott |
|
Jun 2, 2019
| https://github.com/nodejs/node/pull/28033#pullrequestreview-244637247 |
|
View reviewed changes
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| Trott | https://github.com/Trott |
| https://github.com/nodejs/node/pull/28033#pullrequestreview-244637247 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| https://github.com/Trott |
| Trott | https://github.com/Trott |
|
Jun 2, 2019
| https://github.com/nodejs/node/pull/28033#pullrequestreview-244637395 |
|
View reviewed changes
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| src/node_messaging.cc | https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22#diff-512ab8348327408c9a5ba3f7c60d8f8977a672a22b96345b8cfc2562936062f5 |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| https://github.com/lpinca |
| lpinca | https://github.com/lpinca |
|
Jun 2, 2019
| https://github.com/nodejs/node/pull/28033#pullrequestreview-244640245 |
|
View reviewed changes
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| https://github.com/cjihrig |
| cjihrig | https://github.com/cjihrig |
|
Jun 2, 2019
| https://github.com/nodejs/node/pull/28033#pullrequestreview-244643270 |
|
View reviewed changes
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| https://github.com/TimothyGu |
| TimothyGu | https://github.com/TimothyGu |
|
Jun 3, 2019
| https://github.com/nodejs/node/pull/28033#pullrequestreview-245033673 |
|
View reviewed changes
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| src/node_messaging.cc | https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22#diff-512ab8348327408c9a5ba3f7c60d8f8977a672a22b96345b8cfc2562936062f5 |
| TimothyGu | https://github.com/TimothyGu |
| Jun 3, 2019 | https://github.com/nodejs/node/pull/28033#discussion_r289964991 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| https://html.spec.whatwg.org/multipage/web-messaging.html#messageport | https://html.spec.whatwg.org/multipage/web-messaging.html#messageport |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| addaleax | https://github.com/addaleax |
| Jun 9, 2019 | https://github.com/nodejs/node/pull/28033#discussion_r291844032 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| @TimothyGu | https://github.com/TimothyGu |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| TimothyGu | https://github.com/TimothyGu |
| Jun 10, 2019 | https://github.com/nodejs/node/pull/28033#discussion_r291865394 |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| https://github.com/BridgeAR |
| BridgeAR | https://github.com/BridgeAR |
|
Jun 4, 2019
| https://github.com/nodejs/node/pull/28033#pullrequestreview-245392550 |
|
View reviewed changes
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
| addaleax | https://github.com/addaleax |
| June 9, 2019 19:33 | https://github.com/nodejs/node/pull/28033#commits-pushed-5c9f1c6 |
|
| https://github.com/addaleax |
|
| https://github.com/Trott |
| fixup! worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/pull/28033/commits/5c9f1c69b29a026c89f1fd7ac0d9f64c4e1c185c |
| 5c9f1c6 | https://github.com/nodejs/node/pull/28033/commits/5c9f1c69b29a026c89f1fd7ac0d9f64c4e1c185c |
|
| https://github.com/addaleax |
| fixup! worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/pull/28033/commits/8aac5b20816d42a4935fb2f2d145963d5bf48eb9 |
| 8aac5b2 | https://github.com/nodejs/node/pull/28033/commits/8aac5b20816d42a4935fb2f2d145963d5bf48eb9 |
|
| https://github.com/addaleax |
| fixup! worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/pull/28033/commits/ace5dff006f5fc4cc0cabd64a9116b240be55b49 |
| ace5dff | https://github.com/nodejs/node/pull/28033/commits/ace5dff006f5fc4cc0cabd64a9116b240be55b49 |
| https://github.com/nodejs-github-bot |
| nodejs-github-bot | https://github.com/nodejs-github-bot |
| Jun 9, 2019 | https://github.com/nodejs/node/pull/28033#issuecomment-500231640 |
| https://ci.nodejs.org/job/node-test-pull-request/23730/ | https://ci.nodejs.org/job/node-test-pull-request/23730/ |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| https://github.com/addaleax |
| addaleax | https://github.com/addaleax |
| Jun 10, 2019 | https://github.com/nodejs/node/pull/28033#issuecomment-500423026 |
| 269c1d6 | https://github.com/nodejs/node/commit/269c1d69c957811d7f5054a524df8f151b5c5781 |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
| https://github.com/addaleax |
| addaleax | https://github.com/addaleax |
| Jun 10, 2019 | https://github.com/nodejs/node/pull/28033#event-2401016117 |
| https://github.com/addaleax |
| addaleax | https://github.com/addaleax |
| June 10, 2019 13:48 | https://github.com/nodejs/node/pull/28033#event-2401016338 |
| addaleax | https://github.com/addaleax |
|
Jun 10, 2019
| https://github.com/nodejs/node/pull/28033#ref-commit-269c1d6 |
|
| https://github.com/addaleax |
| worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/commit/269c1d69c957811d7f5054a524df8f151b5c5781 |
| 269c1d6 | https://github.com/nodejs/node/commit/269c1d69c957811d7f5054a524df8f151b5c5781 |
| #28033 | https://github.com/nodejs/node/pull/28033 |
| BridgeAR | https://github.com/BridgeAR |
|
Jun 17, 2019
| https://github.com/nodejs/node/pull/28033#ref-commit-79a8cd0 |
|
| https://github.com/addaleax |
|
| https://github.com/BridgeAR |
| worker: add typechecking for postMessage transfer list | https://github.com/nodejs/node/commit/79a8cd0dec63f9da7b829cafcaaeb4428d86ec7a |
| 79a8cd0 | https://github.com/nodejs/node/commit/79a8cd0dec63f9da7b829cafcaaeb4428d86ec7a |
| #28033 | https://github.com/nodejs/node/pull/28033 |
| https://github.com/BridgeAR |
| BridgeAR | https://github.com/BridgeAR |
|
Jun 17, 2019
| https://github.com/nodejs/node/pull/28033#ref-pullrequest-457113206 |
|
v12.5.0 proposal
#28268
| https://github.com/nodejs/node/pull/28268 |
| https://github.com/addaleax |
| addaleax | https://github.com/addaleax |
|
Aug 25, 2019
| https://github.com/nodejs/node/pull/28033#ref-pullrequest-484968987 |
|
worker: make transfer list behave like web MessagePort
#29319
| https://github.com/nodejs/node/pull/29319 |
| addaleax | https://github.com/addaleax |
|
Aug 25, 2019
| https://github.com/nodejs/node/pull/28033#ref-commit-440f2b2 |
|
| https://github.com/addaleax |
| worker: make transfer list behave like web MessagePort | https://github.com/addaleax/node/commit/440f2b25bb74dd6e22fec74d8444a61c2b62d799 |
| 440f2b2 | https://github.com/addaleax/node/commit/440f2b25bb74dd6e22fec74d8444a61c2b62d799 |
| nodejs#28033 (comment) | https://github.com/nodejs/node/pull/28033#discussion_r289964991 |
| danbev | https://github.com/danbev |
|
Sep 6, 2019
| https://github.com/nodejs/node/pull/28033#ref-commit-72650bc |
|
| https://github.com/addaleax |
|
| https://github.com/danbev |
| worker: make transfer list behave like web MessagePort | https://github.com/nodejs/node/commit/72650bcf7283c7b9eb9df9154f981eabdf8c523a |
| 72650bc | https://github.com/nodejs/node/commit/72650bcf7283c7b9eb9df9154f981eabdf8c523a |
| #29319 | https://github.com/nodejs/node/pull/29319 |
| #28033 (comment) | https://github.com/nodejs/node/pull/28033#discussion_r289964991 |
| targos | https://github.com/targos |
|
Sep 20, 2019
| https://github.com/nodejs/node/pull/28033#ref-commit-d2b0568 |
|
| https://github.com/addaleax |
|
| https://github.com/targos |
| worker: make transfer list behave like web MessagePort | https://github.com/nodejs/node/commit/d2b05688901c39342aaa419fd2cb277c1c018671 |
| d2b0568 | https://github.com/nodejs/node/commit/d2b05688901c39342aaa419fd2cb277c1c018671 |
| #29319 | https://github.com/nodejs/node/pull/29319 |
| #28033 (comment) | https://github.com/nodejs/node/pull/28033#discussion_r289964991 |
| https://github.co/hiddenchars |
| https://github.com/nodejs/node/pull/{{ revealButtonHref }} |
| Sign up for free | https://github.com/join?source=comment-repo |
| Sign in to comment | https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fpull%2F28033 |
|
| https://github.com/Trott |
|
Trott
| https://github.com/Trott |
|
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
|
| https://github.com/lpinca |
|
lpinca
| https://github.com/lpinca |
|
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
|
| https://github.com/cjihrig |
|
cjihrig
| https://github.com/cjihrig |
|
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
|
| https://github.com/BridgeAR |
|
BridgeAR
| https://github.com/BridgeAR |
|
| https://github.com/nodejs/node/pull/28033/files/7580ecb7a1e8efff7345f27288e77b794787ea22 |
|
| https://github.com/TimothyGu |
|
TimothyGu
| https://github.com/TimothyGu |
|
| https://github.com/nodejs/node/pull/28033/files/5c9f1c69b29a026c89f1fd7ac0d9f64c4e1c185c |
|
worker
| https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Aworker |
| Please reload this page | https://github.com/nodejs/node/pull/28033 |
|
| https://github.com/addaleax |
|
| https://github.com/nodejs-github-bot |
|
| https://github.com/Trott |
|
| https://github.com/lpinca |
|
| https://github.com/TimothyGu |
|
| https://github.com/cjihrig |
|
| https://github.com/BridgeAR |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |