| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -0,0 +1,303 @@ | |||
| 1 | + # Node Foundation CTC Meeting 2016-09-28 | ||
| 2 | + | ||
| 3 | + ## Links | ||
| 4 | + | ||
| 5 | + * **Audio Recording**: TBP | ||
| 6 | + * **GitHub Issue**: [#8802](https://github.com/nodejs/node/issues/8802) | ||
| 7 | + * **Minutes Google Doc**: <https://docs.google.com/document/d/1ZsLMM29guCBPGh7KhuhZEq6YQCKybiTfTUIXajTBalM/> | ||
| 8 | + * _Previous Minutes Google Doc_: <https://docs.google.com/document/d/1-PS20PGCJQIi4_quBHqVgrQNJeJP10NH21wUelcMhfw> | ||
| 9 | + | ||
| 10 | + | ||
| 11 | + ## Present | ||
| 12 | + | ||
| 13 | + * Anna Henningsen @addaleax (CTC) | ||
| 14 | + * Сковорода Никита Андреевич @ChALkeR (CTC) | ||
| 15 | + * Colin Ihrig @cjihrig (CTC) | ||
| 16 | + * Evan Lucas @evanlucas (CTC) | ||
| 17 | + * Jeremiah Senkpiel @Fishrock123 (CTC) | ||
| 18 | + * Tracy Hinds @hackygolucky (observer/Node.js Foundation) | ||
| 19 | + * Josh Gavant @joshgav (observer/Microsoft) | ||
| 20 | + * Michael Dawson @mhdawson (CTC) | ||
| 21 | + * Ali Ijaz Sheikh @ofrobots (CTC) | ||
| 22 | + * Jenn Turner @renrutnnej (observer/Node.js Foundation) | ||
| 23 | + * Rod Vagg @rvagg (CTC) | ||
| 24 | + * Seth Thompson @s3ththompson (observer/Google) | ||
| 25 | + * Myles Borins @TheAlphaNerd (CTC) | ||
| 26 | + * Trevor Norris @trevnorris (CTC) | ||
| 27 | + * Rich Trott @Trott (CTC) | ||
| 28 | + | ||
| 29 | + | ||
| 30 | + ## Standup | ||
| 31 | + | ||
| 32 | + * Anna Henningsen @addaleax (CTC) | ||
| 33 | + * The usual, issues and PR reviews | ||
| 34 | + * Сковорода Никита Андреевич @ChALkeR (CTC) | ||
| 35 | + * Some issue and PR comments and reviews as usual. | ||
| 36 | + * Some more work on docs linting. | ||
| 37 | + * Colin Ihrig @cjihrig (CTC) | ||
| 38 | + * Reviewing issues and PRs. | ||
| 39 | + * Evan Lucas @evanlucas (CTC) | ||
| 40 | + * v6.7.0 release | ||
| 41 | + * More work on types eps | ||
| 42 | + * Jeremiah Senkpiel @Fishrock123 (CTC) | ||
| 43 | + * Issue / PR Review … general stuff | ||
| 44 | + * Working towards ES Modules prototype implementations with Chris Dickinson | ||
| 45 | + * Tracy Hinds @hackygolucky (observer/Node.js Foundation) | ||
| 46 | + * getting the Outreachy info on website | ||
| 47 | + * Josh Gavant @joshgav (observer/Microsoft) | ||
| 48 | + * helping bring in some new MS contributors | ||
| 49 | + * scheduled diag meeting for next week | ||
| 50 | + * Michael Dawson @mhdawson (CTC) | ||
| 51 | + * Finishing off PPC migration | ||
| 52 | + * Fixing AIX issues when building from node-private | ||
| 53 | + * Some work on ABI-stable node | ||
| 54 | + * Misc PR review/lands | ||
| 55 | + * Keeping up with issues | ||
| 56 | + * Post-mortem nodereport review | ||
| 57 | + * Brian White @mscdex (CTC) | ||
| 58 | + * Worked on various performance improvements in node core | ||
| 59 | + * Reviewed PRs, commented on issues | ||
| 60 | + * Ali Ijaz Sheikh @ofrobots (CTC) | ||
| 61 | + * Looking at node+V8 (5.5) integration build failures that seem related to recent parser improvements | ||
| 62 | + * Investigating performance with the new interpreter | ||
| 63 | + * Working with @matthewloring on FFI | ||
| 64 | + * Jenn Turner @renrutnnej (observer/Node.js Foundation) | ||
| 65 | + * No update, just observing | ||
| 66 | + * Rod Vagg @rvagg (CTC) | ||
| 67 | + * Security releases, supposed to be on vacation | ||
| 68 | + * Seth Thompson @s3ththompson (observer/Google) | ||
| 69 | + * async/await landed in V8 Tip of Tree. on track to ship with V8 5.5 | ||
| 70 | + * expect a doc from V8 language team on promise hook API to allow microtask introspection in the near future | ||
| 71 | + * Myles Borins @TheAlphaNerd (CTC) | ||
| 72 | + * issue / pr review | ||
| 73 | + * helping with security release | ||
| 74 | + * backporting inspector | ||
| 75 | + * auditing v4 backlog | ||
| 76 | + * really have to get to that tap reporter | ||
| 77 | + * coming up with outreachy mentor project | ||
| 78 | + * Trevor Norris @trevnorris (CTC) | ||
| 79 | + * AsyncHooks | ||
| 80 | + * Rich Trott @Trott (CTC) | ||
| 81 | + * mentoring more first-time contributors (via Node Todo) | ||
| 82 | + * doc, test PRs | ||
| 83 | + * ramping up a tiny bit on Build WG stuff, but just a tiny bit | ||
| 84 | + | ||
| 85 | + | ||
| 86 | + ## Agenda | ||
| 87 | + | ||
| 88 | + Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. | ||
| 89 | + | ||
| 90 | + ### nodejs/CTC | ||
| 91 | + | ||
| 92 | + * Scheduling Meetings [#14](https://github.com/nodejs/CTC/issues/14) | ||
| 93 | + | ||
| 94 | + ### nodejs/node | ||
| 95 | + | ||
| 96 | + * meta: update NODE_MODULE_VERSION to 51 [#8808](https://github.com/nodejs/node/pull/8808) | ||
| 97 | + * General v7.0.0 / v6 LTS Planning / Discussion | ||
| 98 | + | ||
| 99 | + | ||
| 100 | + ## Previous Meeting Review | ||
| 101 | + | ||
| 102 | + * deps: update V8 to 5.4 [#8317](https://github.com/nodejs/node/pull/8317) | ||
| 103 | + * Scheduling Meetings [#14](https://github.com/nodejs/CTC/issues/14) | ||
| 104 | + * Decide on what problem points for ES Modules we care about the most. [#15](https://github.com/nodejs/CTC/issues/15) | ||
| 105 | + | ||
| 106 | + | ||
| 107 | + ## Minutes | ||
| 108 | + | ||
| 109 | + ### Scheduling Meetings [ctc#14](https://github.com/nodejs/CTC/issues/14) | ||
| 110 | + | ||
| 111 | + @trott: This is a status report. Initial proposal was to dive in and start rotating meetings. Some were on board, some were concerned. Nikita started Google spreadsheet to figure things out. | ||
| 112 | + | ||
| 113 | + One proposal is to move back one hour (12pm Pacific) which would be a mild improvement for Ben and Nikita. | ||
| 114 | + | ||
| 115 | + Input received from NA and EU but not Asia and Australia. Once we have that information we can figure out what might work. | ||
| 116 | + | ||
| 117 | + --- | ||
| 118 | + | ||
| 119 | + ### meta: update NODE_MODULE_VERSION to 51 [#8808](https://github.com/nodejs/node/pull/8808) | ||
| 120 | + | ||
| 121 | + `process.versions.modules` == 48 for v6.7.0. | ||
| 122 | + | ||
| 123 | + Set in build script, we bump this number for each semver-minor. We’d update to 49, but Electron has been bumping in between, so we need to go to 51. | ||
| 124 | + | ||
| 125 | + @thealphanerd: Had a way to do this in the past, but never landed a version of V8 on master [before a release]. So those using master cannot rely on this check. | ||
| 126 | + | ||
| 127 | + Proposal is to add this to master now and v7.x when released. But should we wait to bump till the actual release? | ||
| 128 | + | ||
| 129 | + @rvagg: If you’re using master, it’s been a bit “buyer beware” in the past. | ||
| 130 | + | ||
| 131 | + @thealphanerd: Node-pre-gyp uses the module version number to determine whether to pull the pre-built binary or to rebuild. So that’s causing problems. | ||
| 132 | + | ||
| 133 | + @ofrobots: Is there a disadvantage to doing this now? | ||
| 134 | + | ||
| 135 | + @rvagg: Doesn’t seem to be. | ||
| 136 | + | ||
| 137 | + @joshgav: Any concern that we’d have to bump again at v7.x release? | ||
| 138 | + | ||
| 139 | + @rvagg: We’ll just bump again. | ||
| 140 | + | ||
| 141 | + @addaleax: We should watch what Electron is doing cause they pull in every V8 version. | ||
| 142 | + | ||
| 143 | + @trevnorris: Could it happen that newer version of V8 has a lower module version number? | ||
| 144 | + | ||
| 145 | + @trott: if module version mapped to V8 version we could always be in sync. | ||
| 146 | + | ||
| 147 | + @ofrobots: Problem is that ABI is more than just V8. Also, we’re moving to a VM-neutral API/ABI in the future and that will remove the relationship to a V8 version. | ||
| 148 | + | ||
| 149 | + @rvagg: We should coordinate with Electron and draw from the same pool of numbers. | ||
| 150 | + | ||
| 151 | + @ofrobots: A good point for a bump would be when we bring a new V8 into master. | ||
| 152 | + | ||
| 153 | + @rvagg: This would make testing those nightlies easier. | ||
| 154 | + | ||
| 155 | + @trevnorris: Sounds to me that we can’t map reliably map a version of Node to a version of V8. So pulling from the same pool as Electron might be misleading to developers. If the number in Electron doesn’t match a Node version there would be a conflict. | ||
| 156 | + | ||
| 157 | + @rvagg: People are tracking which module versions map to what, so they could follow this too. | ||
| 158 | + | ||
| 159 | + @trevnorris: Maybe we can give Electron the minor numbers. | ||
| 160 | + | ||
| 161 | + @rvagg: NW.js also had a similar issue. | ||
| 162 | + | ||
| 163 | + @thealphanerd: Electron bumped to 5.1 in an ABI-breaking way, so we have two versions of Node ABIs out there, cause they needed to stay closer to Chromium. | ||
| 164 | + | ||
| 165 | + @rvagg: They don’t need to keep up with the latest version of V8. | ||
| 166 | + | ||
| 167 | + @thealphanerd: Let’s talk offline. | ||
| 168 | + | ||
| 169 | + @rvagg: Back to GitHub? Or do we need to decide now? | ||
| 170 | + | ||
| 171 | + @thealphanerd : PR has a lot of LGTMs, would like to see this land today or tomorrow so we can unbreak master. | ||
| 172 | + | ||
| 173 | + @ofrobots: Two points - one, what to do now; two, what to do going forward? | ||
| 174 | + | ||
| 175 | + Ali and Myles will work on a policy going forward. | ||
| 176 | + | ||
| 177 | + @rvagg: Might belong in LTS repo as we’ve been doing a lot of versioning stuff there. | ||
| 178 | + | ||
| 179 | + **Next steps**: | ||
| 180 | + | ||
| 181 | + * If there are objections raise them in the issue, otherwise ready to merge. | ||
| 182 | + | ||
| 183 | + --- | ||
| 184 | + | ||
| 185 | + ### General v7.0.0 / v6 LTS Planning / Discussion | ||
| 186 | + | ||
| 187 | + @Fishrock123: Make sure all are in the loop. | ||
| 188 | + | ||
| 189 | + Throw v0.10 in too since it’s end of life at end of October. | ||
| 190 | + | ||
| 191 | + @rvagg: LTS map says *first* of October. Some people expect that cause the docs say that. | ||
| 192 | + | ||
| 193 | + @Fishrock123: We discussed keeping it alive till end of December like v0.12, cause that’s when OpenSSL is EOL’ed. | ||
| 194 | + | ||
| 195 | + @mhdawson: If the doc says Oct 1 what’s the downside to sticking with that? | ||
| 196 | + | ||
| 197 | + @rvagg: We may have communicated Oct 31 through some channels, so some people may expect that. | ||
| 198 | + | ||
| 199 | + Or perhaps when v7.x is first released. | ||
| 200 | + | ||
| 201 | + Having said that, it’s been >2 years, so people have had time to migrate. | ||
| 202 | + | ||
| 203 | + @rvagg: Originally 0.12 was slated for EOL in April 2017, we moved it back because of the OpenSSL issue. | ||
| 204 | + | ||
| 205 | + @Fishrock123: Official LTS policy is target date is when the next release/LTS is cut. That’s usually midway through month. | ||
| 206 | + | ||
| 207 | + @rvagg: Push to LTS WG to resolve ASAP. | ||
| 208 | + | ||
| 209 | + @rvagg: James pushing another beta later this week or early next week. | ||
| 210 | + | ||
| 211 | + @Fishrock123: v7.x is now on semver-major freeze. | ||
| 212 | + | ||
| 213 | + @rvagg: There are some semver-major commits in master which aren’t in v7 release. | ||
| 214 | + | ||
| 215 | + @Fishrock123: Might still need to update? Might have been left out by James intentionally? | ||
| 216 | + | ||
| 217 | + @thealphanerd: Not a ton of semver-major things on master. There are the V8 upgrades (patches), and a move of a method to fs/internal. | ||
| 218 | + | ||
| 219 | + Big one is npm@4 coming through the pipeline 1-2 weeks before Node release, should we include that. | ||
| 220 | + | ||
| 221 | + @rvagg: npm@3 had problems originally so we delayed. Should we do the same for npm@4? | ||
| 222 | + | ||
| 223 | + @addaleax: I’d feel comfortable with landing it. Kat said they aren’t concerned if 4 is included now or not. | ||
| 224 | + | ||
| 225 | + @Fishrock123: Things which are deprecated in v4 will still be deprecated (not removed) in v5, so we could bump all the way to v5 in a later release. | ||
| 226 | + | ||
| 227 | + @rvagg: We don’t have to synchronize all these dates to one, we can be flexible if needed. | ||
| 228 | + | ||
| 229 | + @thealphanerd: If we have a date other than late October it might be a good idea to offer a date. | ||
| 230 | + | ||
| 231 | + @ofrobots: Tentatively Oct 18 is the target stable date for V8 5.4. As tentative as usual, not clear till the last moment. Low chance that V8 will be moving a lot around Oct 18. Haven’t seen this date slip by more than 1-2 days. Very low chance that V8 will destablizie us. | ||
| 232 | + | ||
| 233 | + @thealphanerd: Oct 25 as a tentative date for v7.x? | ||
| 234 | + | ||
| 235 | + @rvagg: Ali and Seth, what’s the risk of setting that date now? | ||
| 236 | + | ||
| 237 | + @ofrobots: Close to Oct 18 I can highlight any potential risk. | ||
| 238 | + | ||
| 239 | + @rvagg: Let’s say that - 25th is tentative date, we’ll communicate if there’s any change. Any objections? (No.) | ||
| 240 | + | ||
| 241 | + That will also be the day we switch v6 to LTS. | ||
| 242 | + | ||
| 243 | + @thealphanerd: Doing release of v6 LTS earlier might be helpful so we have that out of the way for potential v7 issues. | ||
| 244 | + | ||
| 245 | + @rvagg: Discussion on this will move to LTS WG. Join the LTS WG on Monday to discuss. | ||
| 246 | + | ||
| 247 | + @thealphanerd: Could use someone to be responsible for v6 LTS, please volunteer. | ||
| 248 | + | ||
| 249 | + @rvagg: It’s been helpful to have a single person for v4 LTS, but we need to find a model that scales in the future. | ||
| 250 | + | ||
| 251 | + @Fishrock123: Would be helpful to schedule LTS a week earlier to avoid problems. | ||
| 252 | + | ||
| 253 | + --- | ||
| 254 | + | ||
| 255 | + ### Supported platforms proposal from Build WG [#488](https://github.com/nodejs/build/issues/488) | ||
| 256 | + | ||
| 257 | + Current proposal: <https://github.com/nodejs/build/issues/488#issuecomment-250155697> | ||
| 258 | + | ||
| 259 | + @trott should make a CTC agenda item next week? | ||
| 260 | + | ||
| 261 | + @rvagg: Give input on that issue before it comes to CTC. Build WG must review and sign off on as well. | ||
| 262 | + | ||
| 263 | + @rvagg: Some discussion about tiers, this affects OS vendors. | ||
| 264 | + | ||
| 265 | + --- | ||
| 266 | + | ||
| 267 | + ### Other | ||
| 268 | + | ||
| 269 | + @thealphanerd: Node.js is going to be working with Outreachy project to help people from underrepresented groups get involved. | ||
| 270 | + | ||
| 271 | + We need projects for these people to work on in 3 months. If you can think of good parts of the project to assign… would love to hear your suggestions. | ||
| 272 | + | ||
| 273 | + @rvagg: GitHub thread? | ||
| 274 | + | ||
| 275 | + @hackygolucky: I’ll create a new one and ping @nodejs/collaborators. | ||
| 276 | + | ||
| 277 | + @rvagg: Are we getting a satisfactory response on the call for mentors? | ||
| 278 | + | ||
| 279 | + @hackygolucky: 5 primary mentors and a number of supplementals. 4 sponsors, which means we can accept 4 mentees. | ||
| 280 | + | ||
| 281 | + @rvagg: If someone wants to be a supplemental is that still open? | ||
| 282 | + | ||
| 283 | + @hackygolucky: Thread is still open: https://github.com/nodejs/education/issues/7 | ||
| 284 | + | ||
| 285 | + --- | ||
| 286 | + | ||
| 287 | + ## Q/A on public channels | ||
| 288 | + | ||
| 289 | + None. | ||
| 290 | + | ||
| 291 | + --- | ||
| 292 | + | ||
| 293 | + ## Upcoming Meetings | ||
| 294 | + | ||
| 295 | + * CTC: 2016-10-05 | ||
| 296 | + * TSC: 2016-10-06 | ||
| 297 | + * Build: 2016-10-11 | ||
| 298 | + * Diagnostics: 2016-10-05, 12pm Pacific | ||
| 299 | + * Benchmarking: | ||
| 300 | + * LTS: 2016-10-03 | ||
| 301 | + * Post-Mortem: | ||
| 302 | + * API: | ||
| 303 | + | ||
| Back | FazBrowse Home | New Git URL |
0 commit comments