fix(ci): add draft check in the workflow yaml

[all-seeing-code]

Problem

Currently the tests runs on draft PR as well which is waste of resources.

Solution

Add a if: ! github.event.pull_request.draft check to prevent GH runs on draft PRs. Reference: https://stackoverflow.com/questions/68349031/only-run-actions-on-non-draft-pull-request

[CLAassistant]

All committers have signed the CLA.

[skrdgraph]

Referencing this issue https://github.com/dgraph-io/projects/issues/14

[skrdgraph]

@anurags92 as I was seeing this, an edge case came to my mind.

• our triggers for CI is a commit*
• a transition event from draft to review on a PR, should that be a trigger as well?

I am not sure if this PR actually forces a run after a transition. It would probably trigger the run when a commit is made on a ready-for-review PR. I am not sure how this works. Leaving my thoughts from observation (I need to read the docs for better understanding)

[all-seeing-code]

@anurags92 as I was seeing this, an edge case came to my mind.

* our triggers for CI is a commit*

* a transition event from draft to review on a PR, should that be a trigger as well?

I am not sure if this PR actually forces a run after a transition. It would probably trigger the run when a commit is made on a ready-for-review PR. I am not sure how this works. Leaving my thoughts from observation (I need to read the docs for better understanding)

See my commit b4feeca which handles exactly this.

[all-seeing-code]

@skrdgraph
By default following events are triggers for a PR:

      - opened
      - reopened
      - synchronize

I have added ready_for_review as well for precisely the reason that you mentioned. It would trigger CI when someone marks their PR as ready to review.

[coveralls]

Coverage remained the same at 37.18% when pulling 995992a on anurags92/draft-pr-no-tests into c6ca6bd on main.