Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make 'workflow' input optional #178

Merged
merged 1 commit into from Jul 25, 2022
Merged

Conversation

amezin
Copy link
Contributor

@amezin amezin commented Jul 25, 2022

run_ids are unique across workflows. So workflow id can be inferred from run_id.

And when neither run_id nor workflow are specified, current workflow can be used as a fallback.

Because workflow was previously a required input, this change shouldn't break any existing pipeline.

@dawidd6 dawidd6 merged commit bd33ca5 into dawidd6:master Jul 25, 2022
@dawidd6
Copy link
Owner

dawidd6 commented Jul 25, 2022

Thanks

@amezin
Copy link
Contributor Author

amezin commented Jul 26, 2022

I think #114 can be closed now

@maximkoretskiy
Copy link

@dawidd6, could you please bump the action version?

Thank you for the project!

@dawidd6
Copy link
Owner

dawidd6 commented Aug 9, 2022

@maximkoretskiy done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants