Skip to content

Commit

Permalink
ci: update the archive command (#67)
Browse files Browse the repository at this point in the history
* ci: update the archive command to `tar`

* fix: update how the tarball is stored as

* ci: trigger image build

* ci: lock the app to v5.3.0

* ci: add `ADD` to Dockerfile

* ci: remove the debugging bit
  • Loading branch information
JoshuaLicense authored Apr 18, 2024
1 parent 144fe55 commit d407d1a
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/cd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ jobs:
with:
project: ${{ matrix.project }}
should-upload-artefact: true
artefact-name: app-${{ matrix.project}}-${{ needs.get-version.outputs[matrix.project] }}
artefact-name: ${{ matrix.project}}
retention-days: 1
permissions:
contents: read
Expand Down Expand Up @@ -209,7 +209,7 @@ jobs:
with:
project: ${{ matrix.project }}
version: ${{ needs.get-version.outputs[matrix.project] }}
app-artefact-name: app-${{ matrix.project}}-${{ needs.get-version.outputs[matrix.project] }}
app-artefact-name: ${{ matrix.project}}
should-upload-artefact-to-ecr: true
permissions:
contents: read
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ jobs:
with:
project: ${{ matrix.project }}
should-upload-artefact: ${{ !!needs.orchestrator.outputs[format('should-build-{0}-docker', matrix.project)] }}
artefact-name: app-${{ matrix.project}}-${{ needs.get-version.outputs[matrix.project] }}
artefact-name: ${{ matrix.project}}
retention-days: 1
permissions:
contents: read
Expand Down Expand Up @@ -199,7 +199,7 @@ jobs:
with:
project: ${{ matrix.project }}
version: ${{ needs.get-version.outputs[matrix.project] }}
app-artefact-name: app-${{ matrix.project}}-${{ needs.get-version.outputs[matrix.project] }}
app-artefact-name: ${{ matrix.project}}
should-upload-artefact-to-ecr: false
permissions:
contents: read
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/docker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ jobs:
- uses: actions/download-artifact@v4
with:
name: ${{ inputs.app-artefact-name }}
path: app/${{ inputs.project }}
path: infra/docker/${{ inputs.project }}

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/php.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
inputs:
ref:
type: string
default: 5.3.0
required: false
project:
type: string
Expand Down Expand Up @@ -181,11 +182,11 @@ jobs:
- name: Install dependencies
run: composer install --no-progress --prefer-dist --no-dev --optimize-autoloader
- name: Archive
run: git archive --output=${{ inputs.artefact-name }}.tar.gz HEAD
run: tar -czf ${{ inputs.project }}.tar.gz *
- name: Upload artefact
if: ${{ inputs.should-upload-artefact }}
uses: actions/upload-artifact@v4
with:
name: ${{ inputs.artefact-name }}
path: app/${{ inputs.project }}/${{ inputs.artefact-name }}.tar.gz
path: app/${{ inputs.project }}/${{ inputs.project }}.tar.gz
retention-days: ${{ inputs.retention-days }}

0 comments on commit d407d1a

Please sign in to comment.