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

build: update upload-pages-artifact action to v3 #200

Merged
merged 3 commits into from
Feb 14, 2025

Conversation

naipaka
Copy link
Contributor

@naipaka naipaka commented Feb 7, 2025

🙌 やったこと

  • update upload-pages-artifact action to v3

✍️ やらないこと

  • なし

📸 スクリーンショット

Before After
image image

📝 その他

@naipaka naipaka requested a review from a team as a code owner February 7, 2025 02:24
@naipaka naipaka requested review from riscait and k-nkmr and removed request for a team February 7, 2025 02:24
@@ -59,7 +59,7 @@ jobs:
--minify \
--baseURL "${{ steps.pages.outputs.base_url }}/"
- name: Upload artifact
uses: actions/upload-pages-artifact@v2
uses: actions/upload-pages-artifact@v3
Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://github.com/altive/handbook/actions/runs/13147588015/job/36688963512

Error: This request has been automatically failed because it uses a deprecated version of actions/upload-artifact: v3. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

上記に対応するためにバージョンをあげました!
v3で v4 upload-artifact に対応したようです✍️

https://github.com/actions/upload-pages-artifact/releases/tag/v3.0.0

@@ -47,7 +47,7 @@ jobs:
run: sudo snap install dart-sass
- name: Setup Pages
id: pages
uses: actions/configure-pages@v3
uses: actions/configure-pages@v5
Copy link
Contributor Author

Choose a reason for hiding this comment

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

lint エラーが出たのでこちらも最新バージョンにあげました!

Copy link
Contributor

@k-nkmr k-nkmr left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@riscait riscait left a comment

Choose a reason for hiding this comment

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

ありがとうございます🙌

@naipaka naipaka merged commit 817af19 into main Feb 14, 2025
2 checks passed
@naipaka naipaka deleted the fix-update-upload-artifact-v4 branch February 14, 2025 00:11
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