Skip to content

fix(ci): pin the docker reusable past the syntax fix - #85

Merged
BryanFRD merged 1 commit into
mainfrom
fix/docker-reusable-pin
Aug 15, 2026
Merged

fix(ci): pin the docker reusable past the syntax fix#85
BryanFRD merged 1 commit into
mainfrom
fix/docker-reusable-pin

Conversation

@BryanFRD

Copy link
Copy Markdown
Contributor

The Dockerfile fix in #84 was only half the story: the shared build script itself did not parse, so v0.24.1 still failed with syntax error: unexpected end of file. That is repaired in .github#257; this moves the pin onto it.

Only docker.yml is repinned. The other reusables in this repo stay where they are — the same range carries a FerrFlow action bump that belongs to a deliberate decision about the release flow, not to unbreaking an image build.

Once this lands, the release it cuts is the first version since 0.20.0 to actually produce a container.

0.21.0 through 0.24.0 stay imageless. Their tags contain the Dockerfile that trips hadolint on SC2155, so rebuilding them means rewriting released tags. They are superseded by the version this PR produces, and the chart's appVersion should simply never point at them.

Copilot AI lite review requested due to automatic review settings August 15, 2026 12:22
@BryanFRD
BryanFRD enabled auto-merge (squash) August 15, 2026 12:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ferrfleet ferrfleet Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Clean, minimal pin bump: same new SHA applied consistently to both the build and verify jobs referencing reusable-docker-build.yml. Scope matches the description (only docker.yml touched, other reusables intentionally left alone). Nothing blocking.

@BryanFRD
BryanFRD merged commit b632145 into main Aug 15, 2026
19 checks passed
@BryanFRD
BryanFRD deleted the fix/docker-reusable-pin branch August 15, 2026 12:23
@github-actions

Copy link
Copy Markdown

SonarQube — aucune nouvelle issue

Comparaison entre le projet bac à sable de cette PR et la branche par défaut : SonarQube Community n'analyse pas les PR, ce delta est calculé côté CI. Détail

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