Skip to content

doc: add minutes for meeting july 28th #489

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

doc: add minutes for meeting july 28th #489

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
58 changes: 58 additions & 0 deletions wg-meetings/2021-07-28.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
# Node.js Diagnostics WorkGroup Meeting 2021-07-28

## Links

* **Recording**: https://www.youtube.com/watch?v=qPa90B8ahSQ
* **GitHub Issue**: https://github.com/nodejs/diagnostics/issues/488

## Present

* Stephen Belanger (@Qard)
* Rafael Gonzaga (@rafaelgss)
* Michael Dawson (@mhdawson)
* Gireesh Punathil (@gireeshpunathil)
* Chengzhong Wu (@legendecas)
* Helio Frota (@helio-frota)

## Agenda

## Announcements

* No announcements this week

*Extracted from **diag-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.

### nodejs/node

* async_hooks: multi-tenant promise hook api [#39283](https://github.com/nodejs/node/pull/39283)
* Working to expose promise hook API to userland and then have async hooks
consume that under the covers
* Currently separate top level module promise_hook
* Conflicting opinions if it should be under async_hooks or separate module
* Want to try to make it a SemVer minor so that it can be backported
* Differing opinions on whether adding a new top level module is SemVer major or not
* Ask is for people to take a closer look/comment on issue.

### nodejs/diagnostics

* inactive branches in the repo [#486](https://github.com/nodejs/diagnostics/issues/486)
* Looks all are stale branches came as a side effect or PR process. Should be good to remove.

* Identify async_hooks use cases beyond AsyncLocalStorage [#437](https://github.com/nodejs/diagnostics/issues/437)
* Created a PR to @nodejs/tweet repo for tweeting through Node.js handle.
https://github.com/nodejs/tweet/pull/40
* Also run it through TSC at some point in time.

### nodejs/node-inspect

* update: add blackbox by default [#99](https://github.com/nodejs/node-inspect/pull/99)
* Document how to flip the configuration in both GUI and CLI mode


## Q&A, Other

## Upcoming Meetings

* **Node.js Foundation Calendar**: <https://nodejs.org/calendar>

Click `+GoogleCalendar` at the bottom right to add to your own Google calendar.