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

WG meeting October 2016 #235

Closed
mcollina opened this issue Sep 29, 2016 · 32 comments
Closed

WG meeting October 2016 #235

mcollina opened this issue Sep 29, 2016 · 32 comments

Comments

@mcollina
Copy link
Member

mcollina commented Sep 29, 2016

Hi @nodejs/streams,

I propose the week between the 24th and the 28th for running our wg meeting.

Agenda:

cc @thealphanerd @addaleax @mafintosh @calvinmetcalf @bengl @chrisdickinson @yoshuawuyts

If you are ok with this I will send out a doodle.

@calvinmetcalf
Copy link
Contributor

I can do the 26th and latter

@yoshuawuyts
Copy link

yup, works for me

@mafintosh
Copy link
Member

same

@MylesBorins
Copy link

I'm on the road Oct 14 - Nov 1. I'll try to make it work whenever you pick

@mscdex
Copy link
Contributor

mscdex commented Oct 16, 2016

Would it be possible to discuss #111 yet?

@calvinmetcalf
Copy link
Contributor

@mcollina we have a date ?

@mcollina
Copy link
Member Author

Sorry, nodeconf.eu happened.
I'll fire up a doodle now.

@mcollina
Copy link
Member Author

Here is the doodle: http://doodle.com/poll/w2wt6yn4xrprwggz.

My bad if this was late :(.

@mcollina
Copy link
Member Author

@calvinmetcalf
Copy link
Contributor

its ok I just got back from a holiday in russia

On Wed, Oct 26, 2016 at 1:40 PM Matteo Collina notifications@github.com
wrote:

Here is the doodle: http://doodle.com/poll/w2wt6yn4xrprwggz.

My bad if this was late :(.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#235 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABE4nxB9AcFhaEebC_iBMab3zN0ibOTXks5q35CWgaJpZM4KJpH7
.

@mcollina
Copy link
Member Author

I'll call it for Monday 31st at 1pm CEST.

@mafintosh
Copy link
Member

@mcollina 👍

@calvinmetcalf
Copy link
Contributor

ok I totally didn't realize those times were cest but I should be able to make it though, I may be a smidge tardy

@mcollina
Copy link
Member Author

@calvinmetcalf do you want to move it to a later time? I picked 17:00 because it was the latest you were free.

@calvinmetcalf
Copy link
Contributor

I thought you picked 13:00 cest? which is 7am edt right?

@mcollina
Copy link
Member Author

17:00 CEST, 12:00 EDT.

I can send you a calendar invite if you prefer.

@mafintosh
Copy link
Member

So its 17 CEST now instead?

@mcollina
Copy link
Member Author

oooh, I've selected the right time on the doodle, and completely mistyped here. Today I'm sloppy 😅

@calvinmetcalf
Copy link
Contributor

ok that I can make easily

@addaleax
Copy link
Member

17:00 CEST should work for me too :)

(I am a bit confused/curious though… are there any countries in Europe which won’t be back on CET after sunday?)

@mafintosh
Copy link
Member

@addaleax good point! @mcollina post a UTC time so we don't mess this up haha

@mcollina
Copy link
Member Author

OK. I'm definitely not up to the task 😂😭😱. I can call 16:00 UTC on Monday 31st, but.... feel free to step in and organize this.

@mafintosh
Copy link
Member

@mcollina it isn't your fault timezones are hard :) 16:00 UTC it is. thanks for taking the time to organise a new meeting

@mafintosh
Copy link
Member

Turns out @yoshuawuyts and I already have a meeting at the same time. Anyway we can move this forward a couple of hours? Or move it to Tuesday?

@addaleax
Copy link
Member

Tuesday should work just as well

@mcollina
Copy link
Member Author

I'm ok moving it one hour (or two), later. What time?

@mafintosh
Copy link
Member

Tuesday same time?

@mcollina
Copy link
Member Author

Wednesday same time, 16:00 UTC.

Start a new doodle 😂
Il giorno sab 29 ott 2016 alle 01:13 Mathias Buus notifications@github.com
ha scritto:

Tuesday same time?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#235 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADL4zSIbXV5akvEHEcOuC8zP9A8oU95ks5q4oF_gaJpZM4KJpH7
.

@sonewman
Copy link
Contributor

I can do then... Sorry late arrival.

@calvinmetcalf
Copy link
Contributor

ok sounds good

@mcollina
Copy link
Member Author

mcollina commented Nov 2, 2016

@mcollina
Copy link
Member Author

mcollina commented Nov 2, 2016

Minutes of our meeting:
streams wg october 2016

  • we slipped some things marked as semver-major in the latest releases, more for safety. Things are currently reviewed before release. However, we might want to look for an automated process in the future.
  • we still support old browsers with 1% or less market share, as well as old Node.js versions (down to v0.8). The team goal is to keep supporting those for the time being, and re-evaluate in the next few months.
  • reduce the package size: currently this module is downloaded 40 millions time per month, and if we can speed up the downloads by half a second, we will still save humanity a huge amount of time.
    • vendor dependencies that we use and maintain inline isArray #238 (but maybe is better to use bundleDependencies, as noted afterwards), because we want to keep backward compatibility.
    • remove docs
  • @calvinmetcalf will champion flush() for WritableStream
  • @mafintosh will write an EPS on destroy().

Vendoring readable-stream in core

This was the major topic of the discussion, as we agreed at Interactive EU, we will submit a proposal to have core depend on readable-stream, and not readable-stream pulling from core.

In order to move things forward, I will write an EPS based on the following:

  1. we will maintain the build step, as it currently is
  2. import the code from nodejs/node into nodejs/readable-stream, and base the build from that
  3. vendor readable-stream into core

If things move forward, I would like to present this at the collaborator summit of NINA.

@nodejs/streams, please make edits if things needs changing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants