Bug with pep8ify

[pep8speaks-org]

Bug with pep8ify

[OrkoHunter]

Hey @prakhargurunani ! Thank you very much for looking into this problem. Unfortunately, I am a bit occupied and won't be able to work on it. But I also don't want to be a blocker, so go ahead and experiment as you like. :)

It seems fork_fullname is not defined here. What do you think @OrkoHunter ?

Might be a good idea to dump the object and see if the key is included there. You can start local pep8speaks server and send GitHub-like payloads manually to your local pep8speaks server.

[OrkoHunter]

Duplicate #24 #54

[FirePing32]

https://github.com/OrkoHunter/pep8speaks/blob/837643bb95c18a5364cd0539b0f8edaeb3813a76/pep8speaks/handlers.py#L100-L103

It seems fork_fullname is not defined here. What do you think @OrkoHunter ?

[FirePing32]

https://github.com/OrkoHunter/pep8speaks/blob/837643bb95c18a5364cd0539b0f8edaeb3813a76/pep8speaks/helpers.py#L499-L533

Here, we define ghrequest.fork_fullname for the first time at line 504, and that also inside the fork_for_pr() function. So, it is not globally accessible. Is that the correct reason for the error shown above @OrkoHunter ?

[FirePing32]

@OrkoHunter Any update on this ?

[FirePing32]

Yeah. I will try to figure it out.