Skip to content
This repository was archived by the owner on May 10, 2018. It is now read-only.

Conversation

@Aaron1011
Copy link
Contributor

I'll be creating a corresponding travis-api PR soon.

@rkh
Copy link
Contributor

rkh commented Nov 11, 2014

Doesn't this need to be scoped by repo in that case?

@Aaron1011
Copy link
Contributor Author

@rkh: I don't think so. Builds store their pull_request_number from the Request they're created with.

@rkh
Copy link
Contributor

rkh commented Nov 11, 2014

Yes, but that's not unique, multiple repositories might have a pull request with number 5.

@Aaron1011
Copy link
Contributor Author

Oh, didn't think of that. Thanks!

@Aaron1011
Copy link
Contributor Author

@rkh: It should be fixed now.

@Aaron1011
Copy link
Contributor Author

Ping @rkh

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants