Fix github workflow syntax to run unit-tests on push

[IanVS]

Issue:

Since d3bc320, the unit-tests github workflow has not been running on each merge to next, as it was supposed to.

What I did

I think I had the syntax incorrect for determining whether the workflow was being triggered by a push (or alternatively by a pull_request). This instead uses the github.event_name as shown here: https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#example-using-contexts

I also removed the ${{}} syntax, since it is optional, as mentioned in https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#example-using-contexts:

When you use expressions in an if conditional, you may omit the expression syntax (${{ }}) because GitHub automatically evaluates the if conditional as an expression. For more information, see "Expressions."

How to test

• Is this testable with Jest or Chromatic screenshots?
• Does this need a new example in the kitchen sink apps?
• Does this need an update to the documentation?

😬 I think the only real way to test this is to verify that the workflow does not happen when I open this PR, and then someone should add the ci:matrix label and verify it does run, then the real test comes if/when it is merged, to see if the workflow runs on next.

[nx-cloud]

☁️ Nx Cloud Report

CI ran the following commands for commit e0c0a14. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 1 target

• nx run-many --target=prepare --all --parallel --max-parallel=15

---

Sent with 💌 from NxCloud.

[shilman]

LGTM! 🎖️

[IanVS]

Thanks, looks like it's working as intended now.