Skip to content

Make 'workflow' input optional#178

Merged
dawidd6 merged 1 commit into
dawidd6:masterfrom
amezin:workflow-optional
Jul 25, 2022
Merged

Make 'workflow' input optional#178
dawidd6 merged 1 commit into
dawidd6:masterfrom
amezin:workflow-optional

Conversation

@amezin

@amezin amezin commented Jul 25, 2022

Copy link
Copy Markdown
Contributor

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

dawidd6 commented Jul 25, 2022

Copy link
Copy Markdown
Owner

Thanks

@amezin

amezin commented Jul 26, 2022

Copy link
Copy Markdown
Contributor Author

I think #114 can be closed now

@maximkoretskiy

Copy link
Copy Markdown

@dawidd6, could you please bump the action version?

Thank you for the project!

@dawidd6

dawidd6 commented Aug 9, 2022

Copy link
Copy Markdown
Owner

@maximkoretskiy done.

Ma-ve pushed a commit to Ma-ve/action-download-artifact-with-retry that referenced this pull request Jun 24, 2025
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.

3 participants