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

labels: add subsystem label for API doc #47

Merged
merged 1 commit into from
Jun 21, 2016
Merged

labels: add subsystem label for API doc #47

merged 1 commit into from
Jun 21, 2016

Conversation

mscdex
Copy link
Contributor

@mscdex mscdex commented Jun 21, 2016

This is only done if a PR is only modifying a single subsystem's API documentation.

Feel free to reject this and implement it some other way if you don't like how I did this, I just figured it'd be better to submit some kind of PR instead of just creating a feature request issue.

'doc/api/fs.md'
], false)

t.same(labels, ['doc', 'fs'])

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This is only done if a PR is *only* modifying
a single subsystem's API documentation
@phillipj
Copy link
Member

LGTM. Thumbs up for doing a PR rather than an issue :)

@phillipj phillipj merged commit 2569cee into nodejs:master Jun 21, 2016
@phillipj
Copy link
Member

@mscdex just deployed

@mscdex
Copy link
Contributor Author

mscdex commented Jun 22, 2016

aaaand it works! nodejs/node#7361 (comment)

@mscdex mscdex deleted the add-subsystem-tags-for-docs branch June 22, 2016 07:21
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.

2 participants