Skip to content

Make workflow-trigger an input to download artifact step #104

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

Merged
merged 4 commits into from
Mar 23, 2024

Conversation

jukent
Copy link
Contributor

@jukent jukent commented Mar 21, 2024

I am unsure on why this step to trigger the build-build lives inside of the book-build. Let me know what you think.

E: Thanks @brian-rose for explaining that it doesn't trigger the trigger-book-build.yaml workflow, but tells the book-build action to look in that action for the artifact.

Related to ProjectPythia/projectpythia.github.io#415 and ProjectPythia/projectpythia.github.io#319

@jukent jukent requested a review from brian-rose March 21, 2024 18:11
@@ -75,7 +75,6 @@ jobs:
uses: dawidd6/[email protected]
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
workflow: trigger-book-build.yaml
Copy link
Member

Choose a reason for hiding this comment

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

I think this line is needed to identify the workflow that produced the artifact.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, so it doesn't start a new action. On the portal site, this isn't a workflow though. The workflow that produces the artifact will be in publish-site.yaml. I'll try changing this to be an optional input with the trigger-book-build.yaml as its default value?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That explains why the error on my fork say "Error: no matching workflow run found with any artifacts?"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Any other questions I have about the inputs to this step can be answered in the action-download-artifact documentation 🤦‍♀️

@jukent jukent changed the title rm trigger-build-book step Make workflow trigger an input Mar 21, 2024
@jukent jukent changed the title Make workflow trigger an input Make workflow-trigger an input to download artifact step Mar 22, 2024
Copy link
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

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

This looks sensible to me @jukent !

@jukent jukent merged commit f42725e into ProjectPythia:main Mar 23, 2024
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.

2 participants