diff --git a/.github/workflows/main_matrix.yml b/.github/workflows/main_matrix.yml index 8a25829e4..7280f9df4 100644 --- a/.github/workflows/main_matrix.yml +++ b/.github/workflows/main_matrix.yml @@ -240,6 +240,7 @@ jobs: needs: [build] steps: - uses: actions/checkout@v6 + if: github.event_name == 'pull_request_target' with: filter: blob:none # means we download all the git history but none of the commit (except ones with checkout like the head) fetch-depth: 0 @@ -253,18 +254,20 @@ jobs: uses: actions/upload-artifact@v6 id: upload-manifest with: - name: manifests-all + name: manifests-${{ github.sha }} overwrite: true - path: | - manifests-new/*.mt.json + path: manifests-new/*.mt.json - name: Find the merge base + if: github.event_name == 'pull_request_target' run: echo "MERGE_BASE=$(git merge-base "origin/$base" "$head")" >> $GITHUB_ENV env: base: ${{ github.base_ref }} - head: ${{ github.head_ref }} + head: ${{ github.sha }} - name: Download the old manifests - run: gh run download -R ${{ github.repository }} --commit ${{ env.MERGE_BASE }} --name manifests-all --dir manifest-old/ + if: github.event_name == 'pull_request_target' + run: gh run download -R ${{ github.repository }} --name manifests-${{ env.MERGE_BASE }} --dir manifest-old/ - name: Do scan and post comment + if: github.event_name == 'pull_request_target' run: python3 bin/shame.py ${{ github.event.pull_request.number }} manifests-old/ manifests-new/ release-artifacts: diff --git a/src/modules/PositionModule.h b/src/modules/PositionModule.h index 4a2415058..32e499531 100644 --- a/src/modules/PositionModule.h +++ b/src/modules/PositionModule.h @@ -69,7 +69,7 @@ class PositionModule : public ProtobufModule, private concu // In event mode we want to prevent excessive position broadcasts // we set the minimum interval to 5m const uint32_t minimumTimeThreshold = - max(300000, Default::getConfiguredOrDefaultMs(config.position.broadcast_smart_minimum_interval_secs, 30)); + max(uint32_t(300000), Default::getConfiguredOrDefaultMs(config.position.broadcast_smart_minimum_interval_secs, 30)); #else const uint32_t minimumTimeThreshold = Default::getConfiguredOrDefaultMs(config.position.broadcast_smart_minimum_interval_secs, 30);