Skip to content

ci(all): update actions/upload-artifact on build workflow - #536

Merged
SuperITMan merged 1 commit into
NationalBankBelgium:masterfrom
mhenkens:bugfix/535
Nov 21, 2025
Merged

ci(all): update actions/upload-artifact on build workflow#536
SuperITMan merged 1 commit into
NationalBankBelgium:masterfrom
mhenkens:bugfix/535

Conversation

@mhenkens

Copy link
Copy Markdown
Contributor

actions/upload-artifact@v3 is not supported anymore. We should use actions/upload-artifact@v4
now.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Build fail on github.

Issue Number: #535

What is the new behavior?

Build success

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Other information

`actions/upload-artifact@v3` is not supported anymore.
We should use `actions/upload-artifact@v4`
now.

ISSUES CLOSED: NationalBankBelgium#535
@mhenkens
mhenkens requested a review from SuperITMan November 20, 2025 13:41

@SuperITMan SuperITMan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@SuperITMan
SuperITMan merged commit 3b9e64e into NationalBankBelgium:master Nov 21, 2025
2 checks passed
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