|
| 1 | +# Node.js Diagnostics WorkGroup Meeting 2020-08-19 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: https://youtu.be/8f7qq0UvtbE |
| 6 | +* **GitHub Issue**: https://github.com/nodejs/diagnostics/issues/427 |
| 7 | +* **Minutes HackMD**: https://hackmd.io/GOv4P4DzQ9KN0adi-kImYA?both |
| 8 | + |
| 9 | +## Present |
| 10 | + |
| 11 | +* Stephen Belanger (@qard) |
| 12 | +* Rafael Gonzaga (@rafaelgss) |
| 13 | +* Chengzhong Wu (@legendecas) |
| 14 | +* Mary Marchini (@mmarchini) |
| 15 | + |
| 16 | +## Agenda |
| 17 | + |
| 18 | +## Announcements |
| 19 | + |
| 20 | +*Extracted from **diag-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. |
| 21 | + |
| 22 | +### nodejs/diagnostics |
| 23 | + |
| 24 | +* Expand the scope of deep dive meetings [#417](https://github.com/nodejs/diagnostics/issues/417) |
| 25 | + * Expand the scope beyond journeys |
| 26 | + * No objections |
| 27 | +* Revisit meetings frequency [#426](https://github.com/nodejs/diagnostics/issues/426) |
| 28 | + * Suggestion to alternate in the meeting (at the start we give updates on items on the agenda, afterwards deep dive). |
| 29 | + * Try to keep the topics minimal, remember to remove stalled and non-relevant items |
| 30 | + * Let's try it :) |
| 31 | + * Next week no meeting |
| 32 | + * Sept 2nd meeting: first 15-ish minutes for updates, rest for deep dive |
| 33 | +* discuss: node.js contributor survey [#398](https://github.com/nodejs/diagnostics/issues/398) |
| 34 | + * Not much to discuss, lost traction |
| 35 | + * Removed from agenda until gets traction again |
| 36 | +* \[async_hooks\] stable API - tracking issue [#124](https://github.com/nodejs/diagnostics/issues/124) |
| 37 | + * Fix thenables, ???, broke test for some reason |
| 38 | + * There's a memory safety issue that needs to be fixed |
| 39 | + * Unlikely to happen in real world cases |
| 40 | + * Been working on other stuff, not much progress on async hooks front |
| 41 | + * Personally on the opition that async hooks should never be stable, but should stay on the agenda |
| 42 | + * Preferably it becomes an internal thing and higher level abstractions become stable |
| 43 | +* Improving automated remote connection via Inspector Protcol [#348](https://github.com/nodejs/diagnostics/issues/348) |
| 44 | + * *presented* |
| 45 | + * Can we close the connection after it's opened? |
| 46 | + * not remotely |
| 47 | + * Does it incur overhead while connection is open? |
| 48 | + * No, only when running commands |
| 49 | + * If we add to core, can we close? |
| 50 | + * No |
| 51 | + * What are the steps to add to core? |
| 52 | + * Move repo to foundation, open PR |
| 53 | + * Might face pushback due to dependencies |
| 54 | + |
| 55 | + |
| 56 | +## Q&A, Other |
| 57 | + |
| 58 | +## Upcoming Meetings |
| 59 | + |
| 60 | +* **Node.js Foundation Calendar**: https://nodejs.org/calendar |
| 61 | + |
| 62 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
| 63 | + |
0 commit comments