Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Commit

Permalink
doc: add meeting notes for 2017-04-05
Browse files Browse the repository at this point in the history
PR-URL: #95
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
joshgav committed Apr 14, 2017
1 parent a1ba0da commit 18a0032
Showing 1 changed file with 126 additions and 0 deletions.
126 changes: 126 additions & 0 deletions meetings/2017-04-05.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,126 @@
# Node Foundation CTC Meeting 2017-04-05

## Links

* **Audio recording**: <https://www.youtube.com/watch?v=S69FsEpLacc>
* **GitHub Issue**: [CTC#93](https://github.com/nodejs/CTC/issues/93)
* **Minutes Google Doc**: <https://docs.google.com/document/d/1pJoUGqOfNt4cVjLyPxJzfw3M509R7L-IQfXkHUDrp0Q>
* _Previous Minutes Google Doc: <https://docs.google.com/document/d/1nIXl2UMQs1n-yPencn53MYlfu3RnUltX3MfuaStPzx8>_

## Present

* Сковорода Никита Андреевич @ChALkeR (CTC)
* Colin Ihrig @cjihrig (CTC)
* Evan Lucas @evanlucas (CTC)
* Franziska Hinkelmann @fhinkel (CTC)
* Jeremiah Senkpiel @Fishrock123 (CTC)
* Fedor Indutny @indutny (CTC)
* Josh Gavant @joshgav (observer/TSC)
* Michael Dawson @mhdawson (CTC)
* Brian White @mscdex (CTC)
* Myles Borins @MylesBorins (CTC)
* Ali Ijaz Sheikh @ofrobots (CTC)
* Trevor Norris @trevnorris (CTC)
* Rich Trott @Trott (CTC)

## Standup

* Сковорода Никита Андреевич @ChALkeR (CTC)
* Nothing worth mentioning
* Colin Ihrig @cjihrig (CTC)
* Issues and PRs
* Evan Lucas @evanlucas (CTC)
* Trying to get `crypto.randomFill{Sync}` wrapped up
* Franziska Hinkelmann @fhinkel (CTC)
* Nothing of note
* Jeremiah Senkpiel @Fishrock123 (CTC)
* playing with N-API, misc things
* Fedor Indutny @indutny (CTC)
* working on duk-gyp, port of gyp.js to duktape
* Josh Gavant @joshgav (observer/TSC)
* PRs for debugger and node-inspect items
* Michael Dawson @mhdawson (CTC)
* nodejs-foundation npm user for npm management
* resolve a few build issues on AIX
* A few N-API PRs to fix coverity issues
* Working on N-API documentation
* Starting to look at module tests for N-API ported modules
* Misc reviews/lands
* Brian White @mscdex (CTC)
* Reviewed PRs, commented on issues
* Myles Borins @MylesBorins (CTC)
* Put together meta issue for removal of legacy debug protocol
* Research into TF + I
* Released v6.10.2 and v4.8.2
* Released new version of citgm
* Ali Ijaz Sheikh @ofrobots (CTC)
* Some work on the debug / inspector PRs
* Trevor Norris @trevnorris (CTC)
* AsyncHooks
* Rich Trott @Trott (CTC)
* onboarding Collaborators
* preparing for Code + Learn (Oakland, April 22)

## Agenda

* Remove Legacy Debugger for 8.0.0 [ctc#94](https://github.com/nodejs/CTC/issues/94)
* async\_hooks PR [node#11883](https://github.com/nodejs/node/pull/11883)

## Previous Meeting Review

* buffer: auto random fill Buffer(num) and new Buffer(num) [#11808](https://github.com/nodejs/node/pull/11808)
* buffer: discuss future direction of Buffer constructor API [#9531](https://github.com/nodejs/node/issues/9531)

---

## Minutes

### Remove Legacy Debugger for 8.0.0 [ctc#94](https://github.com/nodejs/CTC/issues/94)

master is on V8 5.7, 5.8 will be promoted to stable right before 8.0.0 release. 5.8 doesn't contain code for legacy debugger. In order to be able to migrate to 5.8 in 8.0.0 lifetime we have to remove support for the old interface now, at beginning of 8.

Even if we start 8.0.0 with 5.7, the Diag WG recommended that we hide the interface by aliasing --debug to --inspect or removing it.

node-inspect CLI debugger has landed in core, invoked via `node inspect script.js`.

Do we need to consider node-inspector? A: No, it's been deprecated per the README (https://github.com/node-inspector/node-inspector).

Some objection because we didn't go through a full deprecation cycle for the Debugger API. Perhaps different because this isn't a Node API. But unless we remove now we'll have to support through entire 8 lifetime.

Based on circumstances, no objections to moving forward with removing the old debugger. Keep listening for feedback from community.

**Next steps**

* Continue with PRs.

---

### async\_hooks PR [node#11883](https://github.com/nodejs/node/pull/11883)

Need more review, hoping to land in an early point release in 8.x.

Review to check for:
1. No regressions.
2. API works correctly.

---

## Q&A, Other

* Q: How can I contribute to Node.js source as a front-end developer? A: Check out nodetodo.org, and participate in Code-and-Learns in Oakland, Ireland, China, and Vancouver, Canada.

---

## Upcoming Meetings

* **Node.js Foundation Calendar**: https://calendar.google.com/calendar/embed?src=kap.co_i17i575te0aes6kaanfjr2e4hs%40group.calendar.google.com

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

* **Collaborators Summit**: May 4-5,
[summit#39](https://github.com/nodejs/summit/issues/39)
* **TC39 Meetup**: May 22,
[CTC#67](https://github.com/nodejs/CTC/issues/67)
* **CTC**: Apr 12, 0500 UTC
* **TSC**: Apr 6, 2000 UTC

0 comments on commit 18a0032

Please sign in to comment.