Skip to content
This repository was archived by the owner on Sep 2, 2023. It is now read-only.

Add Kevin Smith as an observer.#88

Merged
MylesBorins merged 1 commit intonodejs:masterfrom
jdalton:ksmith
May 23, 2018
Merged

Add Kevin Smith as an observer.#88
MylesBorins merged 1 commit intonodejs:masterfrom
jdalton:ksmith

Conversation

@jdalton
Copy link
Copy Markdown
Member

@jdalton jdalton commented May 19, 2018

This adds @zenparsing as an observer.

@ljharb
Copy link
Copy Markdown
Member

ljharb commented May 19, 2018

It'd probably be a good idea to have the individual indicate explicitly that they're interested in a change?

@jdalton
Copy link
Copy Markdown
Member Author

jdalton commented May 19, 2018 •
edited
Loading

Yep, I work with @zenparsing (he's my next door office mate). He attended last week's meeting briefly and has shown interest. I'm just opening the PR to start the process. Kevin is also the Microsoft web platform TC39 rep so you'll be seeing him next week.

@zenparsing
Copy link
Copy Markdown

zenparsing commented May 19, 2018

Thanks, @jdalton! Yes, I believe that the work being done here is crucial to the future of the language and would like to observe.

@jdalton jdalton added the modules-agenda To be discussed in a meeting label May 19, 2018
@benjamingr
Copy link
Copy Markdown
Member

benjamingr commented May 19, 2018

I think Kevin would make an excellent addition

benjamingr

This comment was marked as off-topic.

Sign in to view

GeoffreyBooth

This comment was marked as off-topic.

Sign in to view

xtuc

This comment was marked as off-topic.

Sign in to view

guybedford

This comment was marked as off-topic.

Sign in to view

hybrist

This comment was marked as off-topic.

Sign in to view

@MylesBorins MylesBorins mentioned this pull request May 21, 2018
Closed
MylesBorins

This comment was marked as off-topic.

Sign in to view

benjamn

This comment was marked as off-topic.

Sign in to view

weswigham

This comment was marked as off-topic.

Sign in to view

targos

This comment was marked as off-topic.

Sign in to view

inidaname

This comment was marked as off-topic.

Sign in to view

manekinekko

This comment was marked as off-topic.

Sign in to view

manekinekko

This comment was marked as off-topic.

Sign in to view

@manekinekko
Copy link
Copy Markdown

manekinekko commented May 23, 2018

Sorry, I've approved twice 😅

bmeck

This comment was marked as off-topic.

Sign in to view

ljharb

This comment was marked as off-topic.

Sign in to view

@MylesBorins MylesBorins merged commit 4461feb into nodejs:master May 23, 2018
@GeoffreyBooth GeoffreyBooth removed modules-agenda To be discussed in a meeting labels May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

13 more reviewers

@benjamn benjamn benjamn approved these changes

@ljharb ljharb ljharb approved these changes

@bmeck bmeck bmeck approved these changes

@GeoffreyBooth GeoffreyBooth GeoffreyBooth approved these changes

@MylesBorins MylesBorins MylesBorins approved these changes

@hybrist hybrist hybrist approved these changes

@guybedford guybedford guybedford approved these changes

@benjamingr benjamingr benjamingr approved these changes

@xtuc xtuc xtuc approved these changes

@manekinekko manekinekko manekinekko approved these changes

@targos targos targos approved these changes

@weswigham weswigham weswigham approved these changes

@inidaname inidaname inidaname approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.