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

ci: generate pages artifact in Monty API Docs workflow #74

Merged
merged 5 commits into from
Nov 26, 2024

Conversation

tristanls
Copy link
Contributor

@tristanls tristanls commented Nov 26, 2024

After #73, I learned that the actions/download-artifact@v4 cannot find an artifact uploaded with actions/upload-pages-artifact@v3 (or at least I haven't found a way to configure it correctly).

This pull request uploads a "normal" Monty API Docs artifact via actions/upload-artifact@v4, so that actions/download-artifact@v4 can find it. This "normal" artifact is then downloaded and the GitHub Pages artifact is created and uploaded in the Monty API Docs workflow and then deployed.

@tristanls tristanls changed the title ci: debug artifact upload ci: generate pages artifact in Monty API Docs workflow Nov 26, 2024
@tristanls tristanls marked this pull request as ready for review November 26, 2024 18:33
@tristanls tristanls added infrastructure Changes to infrastructure triaged This issue or pull request was triaged labels Nov 26, 2024
codeallthethingz
codeallthethingz approved these changes Nov 26, 2024
Copy link
Contributor

@codeallthethingz codeallthethingz left a comment

Choose a reason for hiding this comment

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

This seemed like such a simple thing last week and has turned into a marathon. :)

@tristanls
Copy link
Contributor Author

😄 I still don't know if it'll work 🤞.

@tristanls tristanls enabled auto-merge (squash) November 26, 2024 18:41
@tristanls tristanls merged commit 3f00ce5 into thousandbrainsproject:main Nov 26, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Changes to infrastructure triaged This issue or pull request was triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants