Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Extract workflow_run_id from dispatch response#110

Open
rolkool wants to merge 4 commits into
convictional:masterfrom
rolkool:master
Open

Extract workflow_run_id from dispatch response#110
rolkool wants to merge 4 commits into
convictional:masterfrom
rolkool:master

Conversation

@rolkool

@rolkool rolkool commented Dec 19, 2025

Copy link
Copy Markdown

Extracting the workflow run ID from dispatch response

@alvaro-fernandez-zartis

Copy link
Copy Markdown

Is there any way I can try this on my repo?

@rolkool

rolkool commented Dec 19, 2025

Copy link
Copy Markdown
Author

Is there any way I can try this on my repo?

You can as uses: rolkool/trigger-workflow-and-wait@master

@alvaro-fernandez-zartis alvaro-fernandez-zartis left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in my repo and works perfectly. Please approve so that we can merge it 🙏

@rolkool

rolkool commented Dec 19, 2025

Copy link
Copy Markdown
Author

@rolkool

rolkool commented Dec 19, 2025

Copy link
Copy Markdown
Author

Let's pause it for now, as Github rolled-back the change until January 2026

https://github.com/orgs/community/discussions/9752#discussioncomment-15299809

@wvidana

wvidana commented Dec 19, 2025

Copy link
Copy Markdown

Let's pause it for now, as Github rolled-back the change until January 2026

https://github.com/orgs/community/discussions/9752#discussioncomment-15299809

@rolkool isn't this fix necessary before january? before github decides to release their change again

@rolkool

rolkool commented Dec 22, 2025

Copy link
Copy Markdown
Author

Let's pause it for now, as Github rolled-back the change until January 2026
https://github.com/orgs/community/discussions/9752#discussioncomment-15299809

@rolkool isn't this fix necessary before january? before github decides to release their change again

If we merge this, before they rollout again the change, it will affect all users of this action, who use it as @master (which is not a good practice in any case). Also we cannot be sure that they will release it again, with the exact same payload

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants