SharezoneApp / sharezone-app

Sharezone is a collaborative school organization app for iOS, Android, macOS and web with +500,000 downloads. Built with Flutter & Firebase.
https://sharezone.net
European Union Public License 1.2
273 stars 48 forks source link

Disable concurrency for `sz` command in our CI/CD pipeline #1659

Closed nilsreichardt closed 3 months ago

nilsreichardt commented 3 months ago

Not sure if there is another easy way to fix #1658

nilsreichardt commented 3 months ago

It still doesn't fix the issue: https://github.com/SharezoneApp/sharezone-app/actions/runs/9437120612/job/25992640142?pr=1659

nilsreichardt commented 3 months ago
image

@Jonas-Sander Do you know why still 5 packages are processed concurrent while using --max-concurrent-packages 1? https://github.com/SharezoneApp/sharezone-app/actions/runs/9437120612/job/25992640142?pr=1659

Jonas-Sander commented 3 months ago
image

@Jonas-Sander Do you know why still 5 packages are processed concurrent while using --max-concurrent-packages 1? https://github.com/SharezoneApp/sharezone-app/actions/runs/9437120612/job/25992640142?pr=1659

No, it works locally like it should. I'm gonna play around with it in this PR.

Jonas-Sander commented 3 months ago

@nilsreichardt putting it into one line works.

So instead of

        run: sz analyze \
          --max-concurrent-packages 1 \
          --package-timeout-minutes 15

it works like this

        run: sz analyze --max-concurrent-packages 1 --package-timeout-minutes 15
nilsreichardt commented 3 months ago

Ah the | at the beginning is missing 😭