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 per-WG label to pull requests #1

Merged
merged 9 commits into from
Feb 12, 2014

Conversation

dontcallmedom
Copy link
Contributor

This adds the ability to label pull requests with e.g. wg:dap when the group responsible for the relevant spec is the Device APIs Working Group.
It takes the data about Working Groups from specrefs

if (specref && specref[spec]) {
if (specref[spec]["deliveredBy"]) {
wgs = specref[spec]["deliveredBy"];
} else if (specref[spec]["aliasOf"]) {
Copy link
Contributor

Choose a reason for hiding this comment

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

You need to wrap this in a while loop as aliasOf can be point to another alias itself.

var request = require('request'),
q = require('q');

function errFrom(body) {
Copy link
Contributor

Choose a reason for hiding this comment

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

You don't need the above for specref, it doesn't have any specific error codes outside of http 404.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed

tobie added a commit that referenced this pull request Feb 12, 2014
Add per-WG label to pull requests
@tobie tobie merged commit 622af59 into web-platform-tests:master Feb 12, 2014
@tobie
Copy link
Contributor

tobie commented Feb 12, 2014

Turns out I don't have auto-deploy setup here. I'll try to setup deployment tonight. Ping me if you don't hear back from me.

@dontcallmedom
Copy link
Contributor Author

On mer., 2014-02-12 at 03:02 -0800, Tobie Langel wrote:

Turns out I don't have auto-deploy setup here. I'll try to setup
deployment tonight. Ping me if you don't hear back from me.

Ping :)

Dom

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.

None yet

2 participants