Pipeline Verbesserungen #11
Conversation
.github/workflows/build-all.yml
Outdated
| - if: github.ref == 'refs/heads/test-pip' | ||
| uses: it-at-m/.github/.github/actions/action-build-image@pipeline-feedback | ||
| with: | ||
| path: "${{ matrix.app-path }}" | ||
| image-name: "${{ matrix.app-path }}" |
There was a problem hiding this comment.
Ich hätte gerne diesen Step als separaten Job gehabt. Leider weiß ich nicht, wie man dann die Matrix nur einmal definiert hätte.
|
@simonhir gibt es noch Anmerkungen von dir? |
.github/workflows/build-all.yml
Outdated
| outputs: | ||
| matrix: ${{ toJSON(matrix) }} |
There was a problem hiding this comment.
da braucht man nicht mehr
.github/workflows/build-all.yml
Outdated
| - app-path: sps-backend | ||
| enable-node: false | ||
| language: java |
There was a problem hiding this comment.
die language muss man eigentlich gar nicht angeben. man kann ja anhand des dateipfads erkennen, ob es eine package.json oder pom.xml enthält
.github/workflows/build-all.yml
Outdated
| strategy: | ||
| fail-fast: false |
There was a problem hiding this comment.
Da war noch das build image in einem anderen Job. Habe gedacht das es besser ist, wenn das builden überall noch weiter läuft. So sieht man gleich, ob es mehrere Fehler gibt. Ansonsten wird es gleich im Lauf abgebrochen
There was a problem hiding this comment.
Ah ok 👍 besides, anyone can change it if it bothers them
.github/workflows/build-all.yml
Outdated
| outputs: | ||
| matrix: ${{ toJSON(matrix) }} |
|
@simonhir passt das jetzt so. Die Pipeline wird immer komplizierter. Spotless, sonarlint, owasp muss man ja auch noch einbauen |
Description
Pipeline in eine Matrix gebaut und Node ausgelagert
Reference
Issues it-at-m/.github#18