Update PR number logic

[Pandapip1]

Blocking ethereum/EIPs#5328

[Pandapip1]

@MicahZoltu I require your approval. Please merge this.

[MicahZoltu]

Can you help me understand why this is necessary? How do we end up in a situation where context.payload?.pull_request?.number doesn't exist but PR_NUMBER environment variable does?

[Pandapip1]

How do we end up in a situation where context.payload?.pull_request?.number doesn't exist but PR_NUMBER environment variable does?

When the workflow is run manually.

[MicahZoltu]

Hmm, do we have an example of this failing? Even for manual re-runs, I would expect the PR number to be available for any pull_request targeted workflow.

[Pandapip1]

Hmm, do we have an example of this failing? Even for manual re-runs, I would expect the PR number to be available for any pull_request targeted workflow.

Let me be a bit more specific: this is for when we aren't targeting this with pull_request. Currently, this is unnecessary. It is preventing me from making some changes, however.

[MicahZoltu]

Ah, I see. The idea is to trigger a workflow run, but in order to do that we need to parameterize the inputs since that workflow won't be run in the context of a PR?