|
| 1 | +*Jan 27th 2017, 6 PM UTC (9PM Finland, 2PM EST, 11AM PST)* |
| 2 | + |
| 3 | +**Attendees:** |
| 4 | +- Intel: [Gabriel Schulhof](https://github.com/gabrielschulhof) |
| 5 | +- IBM: [Michael Dawson](https://github.com/mhdawson) |
| 6 | +- Microsoft: [Taylor Woll](https://github.com/boingoing), [Jason Ginchereau](https://github.com/jasongin), [Arunesh Chandra](https://github.com/aruneshchandra) |
| 7 | + |
| 8 | +**Topics of Discussion:** |
| 9 | + |
| 10 | +*Reviewed Milestone #4:* |
| 11 | +**Milestone 4 delayed - New ETA 3rd Feb 2017.** |
| 12 | + |
| 13 | +*Reviewed Open PRs:* |
| 14 | + |
| 15 | +@fhinkel: Add comment about own properties https://github.com/nodejs/abi-stable-node/pull/33 |
| 16 | +> We should track this as a discussion on the issues page and close the PR. |
| 17 | +* Opened issue #67 for tracking this. |
| 18 | + |
| 19 | +@sampsongao: Add api calls for supporting node-sqlite3 https://github.com/nodejs/abi-stable-node/pull/58 |
| 20 | +> Put this work item on hold until after Milestone 4 |
| 21 | +
|
| 22 | +@boingoing: Error handling prototype 6.2.0 https://github.com/nodejs/abi-stable-node/pull/59 |
| 23 | +>Discussed adding bit field to error response indicating pending exception. |
| 24 | +>Consensus: |
| 25 | +>* Adding more complexity is unnecessary. |
| 26 | +>* Return value should OK or ErrorCode (which may or maynot have exception info) |
| 27 | +>* If the API returns a non OK response then developer can use another API to get stored exception |
| 28 | +
|
| 29 | +>To be ready for final review EOD 1/27 – ETA for PR to land 1/30 |
| 30 | +
|
| 31 | +@jasongin: Fix incorrect use of scope term (ChakraCore) https://github.com/nodejs/abi-stable-node/pull/65 |
| 32 | +>Waiting for Error handling PR to land to rebase and merge |
| 33 | +
|
| 34 | +@jasongin: Fix incorrect use of scope term (V8) https://github.com/nodejs/abi-stable-node/pull/66 |
| 35 | +> Waiting for Error handling PR to land to rebase and merge |
| 36 | +
|
| 37 | +**Action items for Next week:** |
| 38 | +* Update [Error handling PR](https://github.com/nodejs/abi-stable-node/pull/59) with feedback by EOD Friday 1/27 for final review - @boingoing |
| 39 | +* CR and signoff [Error handling PR](https://github.com/nodejs/abi-stable-node/pull/59) ETA Monday 1/30 – Team |
| 40 | +* [Port Error handling to Node 8.0](https://github.com/nodejs/abi-stable-node/issues/39) branch ETA Tuesday 1/31 - @jasongin |
| 41 | +* [Port Error handling to Node-ChakraCore](https://github.com/nodejs/abi-stable-node/issues/40) branch ETA Tuesday 2/2 - @jasongin |
| 42 | +* Hold [SqlLite3 PR](https://github.com/nodejs/abi-stable-node/pull/58) until after Milestone 4 – @sampsongao |
| 43 | +* Update [nanomsg NAPI port](https://github.com/nodejs/abi-stable-node/issues/56) to use error handling - @sampsongao |
| 44 | + |
0 commit comments