-
Notifications
You must be signed in to change notification settings - Fork 26
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
Failing CI test #178
Comments
#179 got past the build stage, but is now failing in deploy |
Updating to
But the deploy step can't find the artifact:
I don't see any clear documentation in the action repository about updating from v2 -> v3, though I did compare our outputs:
artifact-id: ${{ steps.upload-pages-artifact.outputs.artifact-id }} in the build block... so those are two potential changes we could make, but I don't really want to do so without a better understanding of what's going wrong and what those two changes do. @anissa111 do you use a similar action on other repositories? If so, what have you done elsewhere now that |
Sorry for the slow response, I only work half days Fridays currently. I'm not sure why you're running into issues with this right off the top of my head. I haven't seen any issues on the deployments for other repos I'm watching, but none of them use a similar enough build/deploy for that to surprise me. I have a method for testing deployment actions workflows from a personal fork; I'll see if I can fiddle with the artifact upload/download to get it aligned again. |
Ah! I have a fix working on my fork, I'll submit a PR. |
No worries, sorry to flood your inbox when it was already the weekend :)
Awesome, thanks so much!!! |
Describe the bug
The Github Action to build and deploy the webpage is failing; it looks like
actions/upload-pages-artifact@v2
relies onactions/upload-artifact@v3
which is no longer supportedTo Reproduce
Nothing to reproduce, it all happens on the github VMs
Expected behavior
The "Build and Deploy" workflow should complete successfully
Additional context
I'll open a PR momentarily, I think we just need to change
The text was updated successfully, but these errors were encountered: