Skip to content
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.

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

doc: add minutes for meeting 26 Feb 2020 #361

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
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
60 changes: 60 additions & 0 deletions wg-meetings/2020-02-26.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,60 @@
# Node.js Diagnostics WorkGroup Meeting 2020-02-26

## Links

* **Recording**: https://youtu.be/OY-DMO8USl0
* **GitHub Issue**: https://github.com/nodejs/diagnostics/issues/359

## Present

* Michael Dawson (@mhdawson)
* Gireesh Punathil (@gireeshpunathil)
* Giovanny Gongora (@gioyik)
* Matheus Marchini (@mmarchini)

## Agenda

## Announcements

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

* No announcements this week.

### nodejs/diagnostics

* Future of llnode [#355](https://github.com/nodejs/diagnostics/issues/355)
* Last time agreed to change landing policy. Can land own pull request after 3 days if there
are no reviews.
* Going to close issue

* reportVersion semantics are not defined [#349](https://github.com/nodejs/diagnostics/issues/349)
* We had different suggestions, but converging on single number based on changes
to the format.
* Next step. Wait a few more days if no new objections capture agreement and PR
in core docs.

* Proposal to drive Diagnostics WG initiatives through user journeys [#295](https://github.com/nodejs/diagnostics/issues/295)
* Deep dive scheduled for next meeting slot. Nothing to discuss this week.

* Diagnostics "Best Practices" Guide? [#211](https://github.com/nodejs/diagnostics/issues/211)
* covered under work for deep diver for 295 as well.

* \[async_hooks\] stable API - tracking issue [#124](https://github.com/nodejs/diagnostics/issues/124)
* Quard is not here this time so no update.
* Async Storage API landed in core.
* Gireesh, worthwhile to add to test coverage using our existing http tests, can also
update existing tests to use CLS where other methods needed to be used to
pass context.
* Gireesh to open issue about additional testing from the existing HTTP tests for CLS.


## 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.