cucumber-sp / yandex-music-linux

Linux client of Yandex Music
Other
222 stars 5 forks source link

Sheduled udate not launching build #38

Closed cucumber-sp closed 5 months ago

cucumber-sp commented 5 months ago

@Maks1mS по какой-то причине сегодня обновилось до 5.0.10 файлы но build часть показана как skipped. Сейчас запущу вручную но наверное стоит посмотреть почему так случилось

cucumber-sp commented 5 months ago

Нет, даже сейчас пропускает сборку по какой-то причине

Maks1mS commented 5 months ago

@Maks1mS по какой-то причине сегодня обновилось до 5.0.10 файлы но build часть показана как skipped. Сейчас запущу вручную но наверное стоит посмотреть почему так случилось

Пропустил outputs

Нужно так сделать:

    if: ${{ (github.event_name == 'push') || (needs.update_packages.outputs.new_version) }}

И по аналогии везде там где нужно

Maks1mS commented 5 months ago
diff --git a/.github/workflows/update-build-release.yml b/.github/workflows/update-build-release.yml
index fecb61f..00e854c 100644
--- a/.github/workflows/update-build-release.yml
+++ b/.github/workflows/update-build-release.yml
@@ -43,11 +43,11 @@ jobs:
       commit_long_sha: ${{ steps.commit.outputs.commit_long_sha }}

   build:
-    if: ${{ (github.event_name == 'push') || (needs.update_packages.new_version) }}
+    if: ${{ (github.event_name == 'push') || (github.event_name == 'workflow_dispatch') || (needs.update_packages.outputs.new_version) }}
     needs: update_packages
     uses: ./.github/workflows/build.yml
     with:
-      ref: ${{ (needs.update_packages.commit_long_sha || '') }}
+      ref: ${{ (needs.update_packages.outputs.commit_long_sha || '') }}

   upload-release:
     needs: build
@@ -56,7 +56,7 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
         with:
-          ref: ${{ (needs.build.needs.update_packages.commit_long_sha || '') }}
+          ref: ${{ (needs.build.needs.update_packages.outputs.commit_long_sha || '') }}

       - name: Download artifact
         uses: actions/download-artifact@v4