fix(ci): skip publish step on PRs from forks

[hsheth2]

Fixes a subtle regression from #8515.

Checklist

• The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• Links to related issues (if applicable)
• Tests for the changes have been added/updated (if applicable)
• Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
• For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

[asikowitz]

Looks like those 2 stages are consistently failing

[asikowitz]

Seems like we have another check with different logic:

        if: ${{ needs.setup.outputs.publish != 'true' }}

and we go back and forth on which we use... should prob be consistent here. Not sure if this is what's causing the error though

[hsheth2]

@asikowitz lets see if this tweak does the trick

[asikowitz]

Doesn't look like it

[anshbansal]

#8677 should supersede this. Please re-open if that PR does not work.