fix(ci): don't PR or changelog check for draft PRs

[rvagg]

I was just noticing in #12388 how unnecessary the GHA spam is for a Draft, so this should prevent that until you mark as ready-for-review, I think.

[github-actions]

Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions

[BigLep]

Thanks for improving.

[rvagg]

This is still running the PR check, is that because of pull_request_target?

[github-actions]

Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions

[BigLep]

This is still running the PR check, is that because of pull_request_target?

I'm not quite following. @rjan90 added pull_request_target so it runs when a contributor opens a PR from a fork.

I think the reason you keep getting alerted about this is because of the syncrhonize event (and that its removal hasn't been merged yet).

(I'm no expert here though...)

[rvagg]

My point being - I intentionally broke both rules here, no changelog and bad PR title, and it's a draft. But it's complaining about the PR title being wrong, and not the changelog. My expectation is that because I'm modifying the action itself that it would run my edits, but I'm wondering if pull_request_target means it runs the original action from master rather than my branch, while pull_request is running the version off my branch.

[rjan90]

This is still running the PR check, is that because of pull_request_target?

From my search online, it does seem that pull_request_target runs in the context of the base repository and not your feature branch, which is why it is complaining that the PR title is being wrong.

PR title now matches the required format.

not relevant