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

No push notification for local PRs #498

Open
vmarkovtsev opened this issue Jan 30, 2019 · 4 comments
Open

No push notification for local PRs #498

vmarkovtsev opened this issue Jan 30, 2019 · 4 comments
Labels
bug Something isn't working triage/needs-product-input

Comments

@vmarkovtsev
Copy link
Collaborator

If you edit a file using GitHub UI and then create a PR to the same repository from a new branch, Lookout notifies only a new Review event. Although a new branch appears, there is no Push event.

@carlosms carlosms added the bug Something isn't working label Jan 30, 2019
@se7entyse7en
Copy link
Contributor

@carlosms what to we want to do for this?

@carlosms
Copy link
Contributor

Let's see first if we can reproduce it, for master or any other branch.

@smacker
Copy link
Contributor

smacker commented Mar 1, 2019

I used as example this PR: bblfsh/uast-viewer#89
it is created from a local branch greenkeeper/react-styleguidist-9.0.0

In the events endpoint I see such events:

  • CreateEvent:
id	"9101240193"
type	"CreateEvent"
actor	{…}
repo	{…}
payload	
ref	"greenkeeper/react-styleguidist-9.0.0"
ref_type	"branch"
master_branch	"master"
description	null
pusher_type	"user"
public	true
created_at	"2019-02-20T10:35:08Z"
org	{…}
  • PullRequestEvent:

id | "9101240662"
-- | --
type | "PullRequestEvent"
actor | {…}
repo | {…}
payload |  
action | "opened"
number | 89
pull_request | {…}
public | true
created_at | "2019-02-20T10:35:11Z"
org | {…}

There is no PushEvent indeed. Maybe we should handle CreateEvent as PushEvent when ref_type= branch. Though it doesn't have enough information. So we will have to call some additional endpoints.

@carlosms
Copy link
Contributor

carlosms commented Apr 4, 2019

@marnovo do we want to create a push notification in this case?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage/needs-product-input
Projects
None yet
Development

No branches or pull requests

4 participants