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

Add JS (lib/) subsystem labels #43

Merged
merged 2 commits into from
May 13, 2016

Conversation

Fishrock123
Copy link
Contributor

Depends on #41

Does some.. fancy.. mapping for some of the outliers.

Also limits the labels to 4, and just uses lib / src at 5.

cc @phillipj

[/^lib(\/\w+)?\/(_)?link(ed)?list/, 'timers'],
[/^lib\/\w+\/bootstrap_node/, 'lib / src'],
[/^lib\/\w+\/v8_prof_/, 'tools'],
[/^lib\/\w+\/socket_list/, 'net'],

This comment was marked as off-topic.

@Fishrock123 Fishrock123 force-pushed the labels/lib-subsystem branch from 9849685 to 82a93fa Compare May 12, 2016 20:40
@Fishrock123
Copy link
Contributor Author

Rebased on master.

[/^deps\/([^/]+)/, '$1'],

/* JS subsystems */
// Oddities first

This comment was marked as off-topic.

@phillipj
Copy link
Member

LGTM 👍

@Fishrock123 Fishrock123 force-pushed the labels/lib-subsystem branch from 82a93fa to 35657c0 Compare May 12, 2016 21:56
@Fishrock123
Copy link
Contributor Author

Updated a bit, condensed some of the oddities a bit.

@phillipj phillipj merged commit d8f0a38 into nodejs:master May 13, 2016
@phillipj
Copy link
Member

@Fishrock123 deployed

We're now running v6 in production 👯

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

Successfully merging this pull request may close these issues.

2 participants