Forks problem: add an organisation or owner check to run the action

[bigcat88]

When you update a forked ComfyUI Node, the publishing action is constantly triggered and fails.

Can this if: ${{ github.repository_owner == 'NODE_AUTHOR_OWNER' }} be added to the example and to all PRs that this organisation creates to the ComfyUI Nodes, please?

This will prevent action from trigger and it is a default practice how it is usually done, thanks.

[yoland68]

@snomiao Can you take a look at this?

[snomiao]

let me check

[snomiao]

The repository owner name filter has been added. From now on, when creating new pull requests, "NODE_AUTHOR_OWNER" will be replaced by the repository owner's name. For example, when creating a PR for the snomiao/ComfyNode-Registry-test repository, the publish.yml will look like this:

      if: ${{ github.repository_owner == 'snomiao' }}

This change should prevent the publishing action from being constantly triggered and failing.

References:

• Add Github Action for Publishing to Comfy Registry by robinjhuang · Pull Request #35 · snomiao/ComfyNode-Registry-test

[yoland68]

Nice, so going forward, new custom node repos that get's publish.yml PR will contain the repository_owner line right?

[snomiao]

Nice, so going forward, new custom node repos that get's publish.yml PR will contain the repository_owner line right?

yes