|
| 1 | +# Node.js Technical Steering Committee (TSC) Meeting 2020-05-14 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: https://youtu.be/1GatHafSXBg |
| 6 | +* **GitHub Issue**: https://github.com/nodejs/TSC/issues/863 |
| 7 | + |
| 8 | +## Present |
| 9 | + |
| 10 | +* Beth Griggs @BethGriggs (TSC) |
| 11 | +* Сковорода Никита Андреевич @ChALkeR (TSC) |
| 12 | +* Colin Ihrig @cjihrig (TSC) |
| 13 | +* Daniel Bevenius @danbev (TSC) |
| 14 | +* James Snell @jasnell (TSC) |
| 15 | +* Matteo Collina @mcollina (TSC) |
| 16 | +* Michael Dawson @mhdawson (TSC) |
| 17 | +* Michaël Zasso @targos (TSC) |
| 18 | +* Myles Borins @MylesBorins (TSC) |
| 19 | +* Rich Trott @Trott (TSC) |
| 20 | +* Ruben Bridgewater @BridgeAR (TSC) |
| 21 | +* Sam Roberts @sam-github (TSC) |
| 22 | +* Shelley Vohr @codebytere (TSC) |
| 23 | + |
| 24 | +## Agenda |
| 25 | + |
| 26 | +### Announcements |
| 27 | + |
| 28 | +### CPC and Board Meeting Updates |
| 29 | + |
| 30 | +*Extracted from **tsc-agenda** labeled issues and pull requests from the **nodejs org** prior to the meeting. |
| 31 | + |
| 32 | + |
| 33 | +### nodejs/node |
| 34 | + |
| 35 | +* src,win: Replacement of unsupported versions of Windows runtime exit. [#33108](https://github.com/nodejs/node/pull/33108) |
| 36 | +* Dispute of "block running on EOL Windows versions #31954" - Alternative Suggestion [#33034](https://github.com/nodejs/node/issues/33034) |
| 37 | + * https://github.com/nodejs/node/pull/33176 |
| 38 | + * James, raised concern about env variable, but happy given explanations. |
| 39 | + * In general TSC would be happy with 33176 |
| 40 | + |
| 41 | +* process: Throw exception on --unhandled-rejections=default [#33021](https://github.com/nodejs/node/pull/33021) |
| 42 | + * Matheus mentioned its moving forward well. |
| 43 | + * nothing else to discuss this week. |
| 44 | + |
| 45 | +* [Regression in 14.1.0 - Windows] `stdout` is sometimes empty [#33166](https://github.com/nodejs/node/issues/33166) |
| 46 | + * Matteo: specific combinations of build then run environment needed to |
| 47 | + cause. Seems to be hit in github actions. Problem is that we don’t |
| 48 | + have enough windows expertise. Tracked down to a commit, don’t |
| 49 | + understand why that commit affects the failures seen, but backing |
| 50 | + out resolves the issue. |
| 51 | + * Rich any objection to revert. No |
| 52 | + * Michael, not ideal but seems that revert is the right answer now. |
| 53 | + * Chalker, does failing case use the feature? |
| 54 | + * Matteo, no, but passed through code that is modified by the feature |
| 55 | + * Chalker, maybe we can split the code path so that if you are not |
| 56 | + using the feature the code is not changed. |
| 57 | + * James, has not been able to build an environment where it |
| 58 | + Recreate. |
| 59 | + * Michael Z, can we ask that the change be refactored to avoid |
| 60 | + changes to the module loader as that is most likely were the issue |
| 61 | + * Matteo still recommend revert and then look at suggestion by Michael Z |
| 62 | + * Rich, revert sounds reasonable |
| 63 | + * No concerns with that approach from those in meeting |
| 64 | + * Matteo will revert and then well go from there. |
| 65 | + |
| 66 | +### nodejs/TSC |
| 67 | + |
| 68 | +* Node.js future directions - any interest in online or in person summit? [#797](https://github.com/nodejs/TSC/issues/797) |
| 69 | + * Summit session proposed, taking of agenda. |
| 70 | + * James: possibly related, Tierney opened issue about review contribution process and |
| 71 | + policies. https://github.com/nodejs/TSC/issues/864. Might be worth time doing a |
| 72 | + collaborator survey based on kubernetes survey. |
| 73 | + |
| 74 | +### nodejs/admin |
| 75 | + |
| 76 | +* Approval to enable github pages for node-addon-examples [#503](https://github.com/nodejs/admin/issues/503) |
| 77 | + * On agenda for FYI. |
| 78 | + * James would be ok, for team decided. |
| 79 | + * Rich, would prefer if we have issue admin repo. |
| 80 | + * Sam, I think an admin issue is good practice. They get rubber-stamped, but |
| 81 | + I think some kind of org-wide notification is a good idea. Nobody has ever been told “no” |
| 82 | + unreasonably (to my knowledge) |
| 83 | + * Myles, would be good to have admin issue, don’t necessarily want proliferation of |
| 84 | + different sites. |
| 85 | + * James, instead of policy, have best practices |
| 86 | + |
| 87 | +* Audit Google account access [#389](https://github.com/nodejs/admin/issues/389) |
| 88 | + * Brian has proposal in issue. |
| 89 | + |
| 90 | +## Strategic Initiatives |
| 91 | + |
| 92 | +* Build Resources (@mhdawson) |
| 93 | + * No update since last week. |
| 94 | + |
| 95 | +* V8 Currency (@targos) |
| 96 | + * V8 8.3 landed on master. Backport open for v14.x: https://github.com/nodejs/node/pull/33376 |
| 97 | + * Canary (V8 8.4) compiler error on Visual Studio 2017: https://github.com/nodejs/node-v8/issues/157 |
| 98 | + * Discussion around whether we should drop Visual Studio, Matteo should we just build with |
| 99 | + clang ? |
| 100 | + * Michael Z will ping windows team as we still need it for 14.x that does support VS 2017 |
| 101 | + * James, concern is mostly around add-ons versus Node.js itself |
| 102 | + |
| 103 | +* Modules |
| 104 | + * Landed PR to unflag modules in 12.17 |
| 105 | + * Doing some QA, would be good if people can try out the RC, should be as close as possible |
| 106 | + to what is in 14.x |
| 107 | + * Biggest change being discussed is top level await (it is pretty much stable in chrome). PR |
| 108 | + opened by Gus adding it under a flag, required some changes to the options processing. |
| 109 | + Does need more review of the options parsing changes. Would like to land before |
| 110 | + next weeks 14.x release. Plan is to only land unflagged in Node.js after it is unflagged in |
| 111 | + Chrome. |
| 112 | + |
| 113 | +* Core Promise APIs |
| 114 | + * nothing this week |
| 115 | + |
| 116 | +* QUIC |
| 117 | + * ongoing refactor to make it easier to review, taking a bit of time but making process |
| 118 | + * James: If you need a walkthrough just let me know, happy to take you through it |
| 119 | + * Will be behind compile time flag for quite some time. |
| 120 | + |
| 121 | +## Upcoming Meetings |
| 122 | + |
| 123 | +* **Node.js Foundation Calendar**: https://nodejs.org/calendar |
| 124 | + |
| 125 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
0 commit comments