fix(sync-jira-pr): Remove condition "Sync only PRs without comments"

[tomassebestik]

This solves the problem of unsynchronized PRs from repositories with an active DangerJS for GitHub workflow (GH action).

Danger for GitHub (bot github-actions) almost always creates a comment in a pull request faster than the Jira sync cronjob is triggered. Then the PR is ignored by the sync action because it already contains a comment.

If anyone knows of a compelling reason why this should remain unchanged, please explain (in which case we will close this PR and try to approach it differently).

Tested

0. Danger action enabled, JIRA PR SYNC action without change

   • Test PR - test with MASTER JIRA PR sync action (IDFSYNTEST-87) test-project-bot#8
   • https://jira.espressif.com:8443/projects/IDFSYNTEST/issues/IDFSYNTEST-87
     conclusion:
     - didn't synced, created new JIRA ticket when closing comment
     - this is what we want to solve by this PR

1. Danger action enabled, JIRA PR SYNC action with changes in this PR

   • Test PR - test with updated JIRA PR sync action (IDFSYNTEST-85) test-project-bot#7
   • https://jira.espressif.com:8443/projects/IDFSYNTEST/issues/IDFSYNTEST-85
     conclusion:
     - works - synced on cronjob, didn't created new JIRA ticket on comment
     - this demonstrates behavior in projects where GH action Danger + GH action JIRA are both present

2. Danger action disabled, JIRA PR SYNC action with changes in this PR

   • test: disable Danger, use updated branch for Sync PRS action (IDFSYNTEST-89) test-project-bot#10
   • https://jira.espressif.com:8443/browse/IDFSYNTEST-89
     conclusion:
     - this synced normally (expected), Danger comment doesn't block sync process
     - the change implemented by this PR should not affect projects without a Danger GH action

Related

• MM thread: https://mattermost.espressif.cn:6689/espressif/pl/4seqxnecdinf8kfzx4zswkgsda
• issue spotted in: feat: print exceptions to stderr (ESPTOOL-731) esptool#913

[suda-morris]

IIRC, the extra check not pr.comments was added because we found that when a PR is not synced, there are always no user comments. We use this fact to speed up the PR syncing.

[radimkarnis]

LGTM! Thank you.