Skip to content

Use issues endpoints for pull requests#123

Merged
kohsuke merged 2 commits into
hub4j:masterfrom
rtyley:use-issues-endpoints-for-pull-requests
Sep 4, 2014
Merged

Use issues endpoints for pull requests#123
kohsuke merged 2 commits into
hub4j:masterfrom
rtyley:use-issues-endpoints-for-pull-requests

Conversation

@rtyley
Copy link
Copy Markdown
Contributor

@rtyley rtyley commented Sep 3, 2014

Setting labels and assignee on Pull requests failed silently, because the API endpoint being hit contained '/pulls/' rather than '/issues/'.

"Every pull request is an issue, but not every issue is a pull request. For this reason, “shared” actions for both features, like manipulating assignees, labels and milestones, are provided within the Issues API."

https://developer.github.com/v3/pulls/#labels-assignees-and-milestones

If the test assertion fails, we want the pull-request to be closed anyway.
You can't have more than one pull-request merging a given branch into
another, so leaving the PR hanging will cause subsequent test runs to fail
in setup.
Setting labels and assignee on Pull requests failed silently, because
the API endpoint being hit contained '/pulls/' rather than '/issues/'.

"Every pull request is an issue, but not every issue is a pull request.
For this reason, “shared” actions for both features, like manipulating
assignees, labels and milestones, are provided within the Issues API."

https://developer.github.com/v3/pulls/#labels-assignees-and-milestones
@buildhive
Copy link
Copy Markdown

Kohsuke Kawaguchi » github-api #224 SUCCESS
This pull request looks good
(what's this?)

kohsuke added a commit that referenced this pull request Sep 4, 2014
…uests

Use issues endpoints for pull requests
@kohsuke kohsuke merged commit 09557df into hub4j:master Sep 4, 2014
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.

3 participants