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 intoconvictional:masterfrom
rolkool:master
Open

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

Conversation

@rolkool
Copy link
Copy Markdown

@rolkool rolkool commented Dec 19, 2025

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
Copy link
Copy Markdown
Author

rolkool commented Dec 19, 2025

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

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

Copy link
Copy Markdown

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

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
Copy link
Copy Markdown
Author

rolkool commented Dec 19, 2025

@rolkool
Copy link
Copy Markdown
Author

rolkool commented Dec 19, 2025

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
Copy link
Copy Markdown

wvidana commented Dec 19, 2025

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
Copy link
Copy Markdown
Author

rolkool commented Dec 22, 2025

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