Bug: Appears to have trouble with forked repos

[hannahhoward]

Steps to reproduce:

1. Install funnel on a repo
2. Fork that repo to another person/organization
3. Make a properly rebased and squashed branch on the forked repo
4. Submit a PR from the forked repo to the original

Pretty sure Funnel will fail

See current status on #28 - I believe it's rebased and squashed.

[outofambit]

Yes this true, and something that should be fixed. The cause is that a Scent only captures the user and repo name of the pushed to branch. This works fine for branches in the same repo, but not those in different ones.

I think the solution to this is to modify

funnel/lib/funnel/scent.ex

Line 46 in 5cc5c05

@spec get_scent_from_pull_request(%{}) :: %__MODULE__{}

to capture this extra information from the GitHub webhook and then utilize it in https://github.com/carbonfive/funnel/blob/master/lib/funnel/investigator/strategy.ex