Skip to content

ci: update gha to account for node 16 deprecation#4605

Merged
tonistiigi merged 6 commits into
moby:masterfrom
crazy-max:bump-gha
Feb 1, 2024
Merged

ci: update gha to account for node 16 deprecation#4605
tonistiigi merged 6 commits into
moby:masterfrom
crazy-max:bump-gha

Conversation

@crazy-max
Copy link
Copy Markdown
Member

@crazy-max crazy-max commented Feb 1, 2024

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max crazy-max added area/dependencies Pull requests that update a dependency file area/ci labels Feb 1, 2024
@crazy-max crazy-max marked this pull request as ready for review February 1, 2024 08:35
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
jobs:
validate:
uses: docker/docs/.github/workflows/validate-upstream.yml@main
uses: docker/docs/.github/workflows/validate-upstream.yml@919a9b9104a34a40b30d116529bcce589a544d1c # pin for artifact v4 support: https://github.com/docker/docs/pull/19220
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.

Is this because other branches / dependencies are not yet updated? (sorry perhaps silly question; trying to understand why main wouldn't work if both docs and this repository now support v4)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is to avoid having the workflow failing similar to docker/buildx#2210 (comment) in case artifact actions are bumped to v5 in the future on docs main and would/could therefore be incompatible with current v4.

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.

Ah, it's to pin to the current version (prevent updates); I was reading and thought it was downgrading the version 🤗

@tonistiigi tonistiigi merged commit 5ac0a34 into moby:master Feb 1, 2024
@crazy-max crazy-max deleted the bump-gha branch February 2, 2024 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci area/dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants