Skip to content

Commit 9c75596

Browse files
committed
Optimized check
1 parent c509128 commit 9c75596

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

.github/workflows/lint_pr.yml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -218,12 +218,18 @@ jobs:
218218
219219
summary:
220220
needs: [check_changes, lint]
221-
# Only run if there are Python changes and after the lint job completes (success or failure)
222-
if: ${{ always() && needs.check_changes.outputs.has_python_changes == 'true' }}
221+
# Run summary in all cases, regardless of whether lint job ran
222+
if: ${{ always() }}
223223
runs-on: ubuntu-24.04
224224
steps:
225225
- name: Summarize results
226226
run: |
227227
echo "## Pull Request Lint Results" >> $GITHUB_STEP_SUMMARY
228-
echo "Linting has completed for all Python files changed in this PR." >> $GITHUB_STEP_SUMMARY
229-
echo "See individual job logs for detailed results." >> $GITHUB_STEP_SUMMARY
228+
if [[ "${{ needs.check_changes.outputs.has_python_changes }}" == "true" ]]; then
229+
echo "Linting has completed for all Python files changed in this PR." >> $GITHUB_STEP_SUMMARY
230+
echo "See individual job logs for detailed results." >> $GITHUB_STEP_SUMMARY
231+
else
232+
echo "No Python files were changed in this PR. Linting was skipped." >> $GITHUB_STEP_SUMMARY
233+
fi
234+
echo "" >> $GITHUB_STEP_SUMMARY
235+
echo "⚠️ **Note:** This PR still requires manual approval regardless of linting results." >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)