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
Filter PRs from forks #251
Conversation
… which to generate the changelog
{ | ||
"title": "🙅♂️ Forking PR 🙅♂️", | ||
"number": 1, | ||
"body": "No thanks to forking PRs", | ||
"author": { | ||
"login": "jetersen" | ||
}, | ||
"baseRepository": { | ||
"nameWithOwner": "jetersen/release-drafter-test-repo" | ||
}, | ||
"mergedAt": "2020-05-26T20:33:01Z", | ||
"isCrossRepository": true, | ||
"labels": { | ||
"nodes": [] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Such PRs are successfully filtered.
* Add documentation (#5) @TimonVS | ||
* Update dependencies (#4) @TimonVS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
something went wrong when I created the branch
However the PR with the title 🙅♂️ Forking PR 🙅♂️
is filtered out
owner: 'jetersen', | ||
branch: 'forking', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reason for this was because associated PRs does not immediately appear on the main repo or at least that was the behavior I saw.
This hasn't been tested yet. Will add tests asap.
Fixes: #221