CI: tests obligatoires avant push, abandon du tag :latest sur master

Merge pull request 'CI: tests obligatoires avant push, abandon du tag :latest sur master' (#14 ) from fix/workflow-test-before-push into master
Reviewed-on: #14
2026-06-13 18:38:15 +00:00 · 2026-06-13 14:28:56 -04:00 · 2026-06-13 14:25:55 -04:00 · 2026-06-13 14:25:53 -04:00 · 2026-06-13 14:25:50 -04:00
3 changed files with 22 additions and 23 deletions
@@ -11,7 +11,15 @@ jobs:
        continue-on-error: true
        with:
          dockerfile: Dockerfile
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
+      - uses: docker/setup-buildx-action@d7f5e7f509e45cec5c76c4d5afdd7de93d0b3df5 # v4
+      - run: docker build -t ci-image:${{ github.sha }} .
+      - run: bash tests/test.sh ci-image:${{ github.sha }}
  build-push:
+    needs: [test]
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
@@ -31,7 +39,6 @@ jobs:
        with:
          images: jcabillot/offlineimap
          tags: |
-            type=raw,value=latest
            type=raw,value=${{ steps.get-latest-tag.outputs.tag }}-latest,enable=${{ steps.get-latest-tag.outputs.tag != '' }}
      - uses: docker/build-push-action@f9f3042f7e2789586610d6e8b85c8f03e5195baf # v7.2.0
        with:
@@ -39,4 +46,4 @@ jobs:
          push: true
          tags: ${{ steps.meta.outputs.tags }}
          labels: ${{ steps.meta.outputs.labels }}
-          pull: true
+          pull: true
@@ -18,31 +18,15 @@ jobs:
      - uses: docker/setup-buildx-action@d7f5e7f509e45cec5c76c4d5afdd7de93d0b3df5 # v4
      - run: docker build -t ci-image:${{ github.sha }} .
      - run: bash tests/test.sh ci-image:${{ github.sha }}
-  build-push:
+  build:
    needs: [test]
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
      - uses: docker/setup-buildx-action@d7f5e7f509e45cec5c76c4d5afdd7de93d0b3df5 # v4
-      - uses: docker/login-action@650006c6eb7dba73a995cc03b0b2d7f5ca915bee # v4
-        with:
-          username: ${{ secrets.DOCKERHUB_USERNAME }}
-          password: ${{ secrets.DOCKERHUB_TOKEN }}
-      - id: meta
-        uses: docker/metadata-action@80c7e94dd9b9319bd5eb7a0e0fe9291e23a2a2e9 # v6
-        with:
-          images: jcabillot/offlineimap
-          tags: |
-            type=raw,value=latest
-      - uses: docker/build-push-action@f9f3042f7e2789586610d6e8b85c8f03e5195baf # v7.2.0
-        with:
-          context: .
-          push: true
-          tags: ${{ steps.meta.outputs.tags }}
-          labels: ${{ steps.meta.outputs.labels }}
-          pull: true
+      - run: docker build -t jcabillot/offlineimap:${{ github.sha }} .
  tag:
-    needs: [build-push]
+    needs: [build]
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
@@ -57,4 +41,4 @@ jobs:
          DEFAULT_BUMP: patch
          RELEASE_BRANCHES: master
          WITH_V: true
-          GIT_API_TAGGING: false
+          GIT_API_TAGGING: false
@@ -11,7 +11,15 @@ jobs:
        continue-on-error: true
        with:
          dockerfile: Dockerfile
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
+      - uses: docker/setup-buildx-action@d7f5e7f509e45cec5c76c4d5afdd7de93d0b3df5 # v4
+      - run: docker build -t ci-image:${{ github.sha }} .
+      - run: bash tests/test.sh ci-image:${{ github.sha }}
  build-push:
+    needs: [test]
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
@@ -33,4 +41,4 @@ jobs:
          push: true
          tags: ${{ steps.meta.outputs.tags }}
          labels: ${{ steps.meta.outputs.labels }}
-          pull: true
+          pull: true
Author	SHA1	Message	Date
Sagent	a8eb823120	CI: tests obligatoires avant push, abandon du tag :latest sur master	2026-06-13 18:38:15 +00:00
jcabillot	7d37ef7bcb	Merge pull request 'CI: tests obligatoires avant push, abandon du tag :latest sur master' (#14 ) from fix/workflow-test-before-push into master Reviewed-on: #14	2026-06-13 14:28:56 -04:00
cloudix_mcp_server	88cac8d90c	tag: add test job before build-push PR Checks / hadolint (pull_request) Successful in 7s Details PR Checks / build-test (pull_request) Successful in 13s Details - Add test job (build + run tests) - build-push now needs: [test] - Tests must pass before pushing tagged release to DockerHub	2026-06-13 14:25:55 -04:00
cloudix_mcp_server	ae49ab2f80	cron: add test job before build-push, remove :latest tag - Add test job (build + run tests) - build-push now needs: [test] - Remove type=raw,value=latest tag - Keep type=raw,value=$TAG-latest for the latest versioned tag	2026-06-13 14:25:53 -04:00
cloudix_mcp_server	ceb840bfe5	main: remove Docker push, keep only build+test+tag - Remove docker/login-action, docker/metadata-action, docker/build-push-action - Rename build-push to build (just docker build, no push) - Tag job now depends on build - Push to DockerHub is handled solely by tag.yaml now	2026-06-13 14:25:50 -04:00