Skip to content

BC-8763 workaround for actions/build-push-action@v6#3506

Merged
Loki-Afro merged 1 commit intomainfrom
bc-8763
Jan 20, 2025
Merged

BC-8763 workaround for actions/build-push-action@v6#3506
Loki-Afro merged 1 commit intomainfrom
bc-8763

Conversation

@Loki-Afro
Copy link
Member

@Loki-Afro Loki-Afro commented Jan 20, 2025

as it appears to be incompatible with actions/download-artifact@v4 /cloudposse/github-action-matrix-outputs-read#29 (comment) see docker/build-push-action#1167

in short if one downloads an artifact that is not of type zip it might/will fail to download so most of the times this actions/toolkit#1874 might resolve the whole mess

Short Description

Links to Ticket and related Pull-Requests

https://ticketsystem.dbildungscloud.de/browse/BC-8763
hpi-schul-cloud/version-aggregator#27
hpi-schul-cloud/schulcloud-server#5452
hpi-schul-cloud/schulcloud-client#3561
#3506
hpi-schul-cloud/superhero-dashboard#275
hpi-schul-cloud/shd-client#38
hpi-schul-cloud/schulcloud-calendar#169
hpi-schul-cloud/antivirus_check_service#69
hpi-schul-cloud/version-aggregator#27
hpi-schul-cloud/h5p-staticfiles-server#16
hpi-schul-cloud/tldraw-client#122
hpi-schul-cloud/tldraw-server#50

Changes

Data-security

Deployment

New Repos, NPM packages or vendor scripts

Screenshots of UI changes

Checklist before merging

  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • PO: Any deviation from requirements was agreed with Product-Owner / ticket author / support-team
  • DEV: Every new component is implemented having accessibility in mind (e.g. aria-label, role property)

Notice: Please keep this Pull-Request as a Draft (or add WIP label), until it is ready to be reviewed

as it appears to be incompatible with actions/download-artifact@v4 /cloudposse/github-action-matrix-outputs-read#29 (comment)
see docker/build-push-action#1167

in short if one downloads an artifact that is not of type zip it might/will fail to download so most of the times this actions/toolkit#1874 might resolve the whole mess
@sonarqubecloud
Copy link

@Loki-Afro Loki-Afro merged commit 551daa2 into main Jan 20, 2025
70 checks passed
@Loki-Afro Loki-Afro deleted the bc-8763 branch January 20, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants