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

Fix failing GH pages deploy workflow #215

Merged
merged 4 commits into from
Feb 10, 2025

Conversation

hirasso
Copy link
Member

@hirasso hirasso commented Feb 9, 2025

Description

Closes #214

Fix the failing deploy.yml workflow. We deploy to netlify so that file was a leftover anyways

Drive-By

  • Update browserslist

Checks

  • The PR is submitted to the main branch
  • The code was linted before pushing (npm run lint)
  • All tests are passing (npm run test)

Copy link

netlify bot commented Feb 9, 2025

Deploy Preview for splendorous-kataifi-9ae281 ready!

Name Link
🔨 Latest commit 4ca46cf
🔍 Latest deploy log https://app.netlify.com/sites/splendorous-kataifi-9ae281/deploys/67a9ee53e851290008d4d084
😎 Deploy Preview https://deploy-preview-215--splendorous-kataifi-9ae281.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@hirasso hirasso requested a review from daun February 9, 2025 22:36
@daun
Copy link
Member

daun commented Feb 9, 2025

I think we need this action :) Don't we deploy to GitHub Pages?

I wouldn't know how else to trigger a deploy if not via this action.

@daun
Copy link
Member

daun commented Feb 9, 2025

According the the repo settings, we deploy to GitHub Pages. So this action is still required.

@daun
Copy link
Member

daun commented Feb 9, 2025

Looks like we need to deal with the silly forced upload-artifact upgrade :(

@hirasso
Copy link
Member Author

hirasso commented Feb 9, 2025

How does the netlify preview work then in this pr? :)

@hirasso
Copy link
Member Author

hirasso commented Feb 9, 2025

The repo settings are probably a leftover from jekyll times 👹

@hirasso
Copy link
Member Author

hirasso commented Feb 9, 2025

If not sure, lets sleep over it. I'm a bit tired already and might totally be missing something obvious 😄

@daun
Copy link
Member

daun commented Feb 9, 2025

The previews are on Netlify because GitHub Pages can't do branch previews. At least back then it didn't.

What problem are we trying to solve here? I think we need to fix the action, not delete it, right?

@daun
Copy link
Member

daun commented Feb 9, 2025

Looking at the current DNS records, it seems that the main v4 docs are on GitHub Pages, while the old v3 docs are on Netlify. We had tried deploying multiple GitHub Pages sites, but there can only be one site per repo and we didn't want to split the old docs into their own repo. So we had to use Netlify for that.

@hirasso
Copy link
Member Author

hirasso commented Feb 10, 2025

Ok sorry, it was definitely too late for my brain yesterday 🥱😵‍💫. And thank you for keeping a cool head! I will revert the deletion and simply update the action later today.

Sorry for the noise!

@daun
Copy link
Member

daun commented Feb 10, 2025

No worries, it's definitely very confusing with v4, v3, and branch previews all doing slightly different things 🤖

@hirasso
Copy link
Member Author

hirasso commented Feb 10, 2025

Ready :)

@daun daun changed the title Delete failing GH pages deploy workflow F failing GH pages deploy workflow Feb 10, 2025
@daun daun changed the title F failing GH pages deploy workflow Fix failing GH pages deploy workflow Feb 10, 2025
@daun
Copy link
Member

daun commented Feb 10, 2025

Nice! I've renamed to PR 🙃

@hirasso hirasso merged commit 49feaa8 into master Feb 10, 2025
5 checks passed
@hirasso hirasso deleted the fix/actions/upload-pages-artifact-3 branch February 10, 2025 12:40
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.

[Bug]: actions/upload-artifact deprecated
2 participants