Skip to content

Fix deployment: Extract tar before upload#22

Merged
Kaes3kuch3n merged 2 commits intomasterfrom
production-deployment
Apr 15, 2026
Merged

Fix deployment: Extract tar before upload#22
Kaes3kuch3n merged 2 commits intomasterfrom
production-deployment

Conversation

@Kaes3kuch3n
Copy link
Copy Markdown
Member

@Kaes3kuch3n Kaes3kuch3n commented Apr 15, 2026

The download artifact action uncompresses the artifact, but it doesn't extract the contained tar archive automatically, so we have to do this manually before we can upload the extracted files.

See also output of test run at https://github.com/Coderdojo-Schoeneweide/website/actions/runs/24466742834/job/71496019357

@Kaes3kuch3n Kaes3kuch3n requested a review from majdkt April 15, 2026 16:57
@Kaes3kuch3n Kaes3kuch3n self-assigned this Apr 15, 2026
@majdkt
Copy link
Copy Markdown
Collaborator

majdkt commented Apr 15, 2026

Copilot recommends this to avoid manual untaring:

Use the intended deploy action (recommended): actions/deploy-pages in the deploy job. It understands the Pages artifact format and doesn’t require you to manually untar.

@Kaes3kuch3n
Copy link
Copy Markdown
Member Author

Copilot recommends this to avoid manual untaring:

Use the intended deploy action (recommended): actions/deploy-pages in the deploy job. It understands the Pages artifact format and doesn’t require you to manually untar.

The issue with that is that we want to deploy to a separate repository, so we cannot use the default actions/deploy-pages action. Unfortunately the one we use instead doesn't seem to support automatic untaring :(

@majdkt majdkt self-requested a review April 15, 2026 17:11
@Kaes3kuch3n Kaes3kuch3n merged commit d156ff3 into master Apr 15, 2026
1 check passed
@Kaes3kuch3n Kaes3kuch3n deleted the production-deployment branch April 15, 2026 17: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.

2 participants