refactor(ng-dev): move checking if the pull request is a draft into the merge ready check

[josephperrott]

No description provided.

[thePunderWoman]

This would mean that you could still put action: merge on something and that label would sit there while it's not actually merge ready. The point of the original check was to make sure it does not show up when you run the caretaker check or look at all the items in the merge queue, which even though we'd like to assume people use a saved search, typically is just people viewing all the action: merge PRs. So this wouldn't solve that problem. We should only be putting action: merge on PRs that are essentially already ready to merge or at least have final tests runs in progress.

[josephperrott]

I think you are thinking of a different action, this is not the action that applies/removes labels. This is setting the mergeability status.

The appying/removing labels based on draft status is unaffected by this.

[thePunderWoman]

Ahhhhhh, yes you are correct. Disregard my comment.

[josephperrott]

This PR was merged into the repository by commit 872ab8f.

The changes were merged into the following branches: main

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}