[GH workflow] update `analyze.yml` to avoid noisy notification

[awxiaoxian2020]

When I merge PR into main branch, I always received the notification that the workflow failed like this.

I found the PR workflow is different from main branch. The latter doesn't generate comment indeed but it upload it and should be used the other workflow I metion at start.

The PR workflow:

The main workflow:

[awxiaoxian2020]

react.dev/.github/workflows/analyze_comment.yml

Line 13 in b9af9f5

${{ github.event.workflow_run.event == 'pull_request' &&

It seems not including push and we shouldn't but how we handle with this noisy notification?

[gaearon]

If you have a fix in mind, please send it!

[awxiaoxian2020]

@gaearon The point is that do we need analyze comment on main branch?

[harish-sethuraman]

Yea it is better to have. Because sometimes commits land directly into main rather than PR.

[awxiaoxian2020]

Yea it is better to have. Because sometimes commits land directly into main rather than PR.

We can add conditions, I think. I will look into it on the weekend.

[rickhanlonii]

@awxiaoxian2020 do you find this job useful when I works? I was thinking we just remove it.

[alinkedd]

@rickhanlonii please check out #7667 — there are just a few small one-line fixes