nix/2.home-manager
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

ci: use env in update-maintainers changes summary

Browse source 
Use env variables in the changes summary, to avoid string escape issues
with injecting template strings directly into the script.
This commit is contained in:
Matt Sturgeon 2025-07-02 16:59:38 +01:00 • committed by Austin Horstman
parent be8f7e100f
commit bafcf33687
1 changed files with 5 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

7
.github/workflows/update-maintainers.yml vendored
View file

@ -139,10 +139,13 @@ jobs:
--label "dependencies" \
--label "maintainers"
- name: Summary
env:
has_changes: ${{ steps.check-changes.outputs.has_changes }}
changes: ${{ steps.check-changes.outputs.changes_summary }}
run: |
if [[ "${{ steps.check-changes.outputs.has_changes }}" == "true" ]]; then
if [[ "$has_changes" == "true" ]]; then
echo "✅ Successfully created PR with maintainer updates"
echo "📊 Changes: ${{ steps.check-changes.outputs.changes_summary }}"
echo "📊 Changes: $changes"
else
echo "ℹ️ No changes detected - maintainers list is up to date"
fi