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

strategic-intiatives: adding npm integration and open web standards #535

Closed

Conversation

MylesBorins
Copy link
Contributor

adding new initiatives that I would like to take on

  • integration of npm in node
  • engagement in open web standards

@ljharb
Copy link
Member

ljharb commented Apr 26, 2018

What does “integration into node” mean?

| TSC Governance | [Myles Borins][MylesBorins] | https://github.com/nodejs/TSC/issues/383 |
| New Streams APIs | [James Snell][jasnell] + [Jeremiah Senkpiel][fishrock123] | https://github.com/nodejs/node/pull/16414 |
| V8 Currency | [Michaël Zasso][targos] | |
| npm Integration | [Myles Borins][MylesBorins] | |
Copy link
Member

@richardlau richardlau Apr 26, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming this is #528, perhaps npm Currency (as per the line above for V8), or is something different being referred to here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TBH I was going to send a PR to update the V8 initiative name. I think integration makes more sense.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either name works for me, as long as they are consistent if referring to the same goals.

@MylesBorins
Copy link
Contributor Author

@ljharb integration as in integrating npm into the node.js source tree

@ljharb
Copy link
Member

ljharb commented Apr 26, 2018

@MylesBorins like copypasta, or something like submodules?

| New Streams APIs | [James Snell][jasnell] + [Jeremiah Senkpiel][fishrock123] | https://github.com/nodejs/node/pull/16414 |
| V8 Currency | [Michaël Zasso][targos] | |
| npm Integration | [Myles Borins][MylesBorins] | |
| Open Web Standards | [Myles Borins][MylesBorins] | |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is possible to have at least an issue or something with more details for each one and add a link?

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a suggestion.

@mhdawson
Copy link
Member

mhdawson commented May 9, 2018

Going to land as is, @MylesBorins would be good if you could add links in a follow on PR.

@mhdawson
Copy link
Member

mhdawson commented May 9, 2018

@MylesBorins needs a rebase.

@mhdawson
Copy link
Member

mhdawson commented Jun 6, 2018

@MylesBorins could you rebase?

mhdawson pushed a commit that referenced this pull request Jun 20, 2018
adding `npm integration` and `open web standards`

PR-URL: #535
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@mhdawson
Copy link
Member

merged and landed as 88b45a1

@mhdawson mhdawson closed this Jun 20, 2018
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

Successfully merging this pull request may close these issues.

9 participants