Skip to content

fix(ci): secrets validation fix for nuget.yml and deploy-server-side.yml#374

Merged
csharpfritz merged 1 commit intoFritzAndFriends:mainfrom
csharpfritz:ci-fix-promote
Feb 25, 2026
Merged

fix(ci): secrets validation fix for nuget.yml and deploy-server-side.yml#374
csharpfritz merged 1 commit intoFritzAndFriends:mainfrom
csharpfritz:ci-fix-promote

Conversation

@csharpfritz
Copy link
Copy Markdown
Collaborator

Summary

Cherry-picked CI workflow fix (from #372) onto main. Single clean commit.

Changes

  • nuget.yml: Fixed secrets.* usage in step-level if condition
  • deploy-server-side.yml: Fixed identical secrets.* pattern

Root Cause

GitHub Actions rejects secrets.* in step-level if conditions during workflow validation, causing 0 jobs and "workflow file issue" errors.

Fix

Both files now pass secrets through env: block first, then reference env.VAR_NAME in the if condition.

After Merge

  • deploy-server-side.yml will successfully push container images on next main push
  • nuget.yml will work correctly on next tag push

GitHub Actions rejects secrets.* in step-level if conditions during
workflow validation, causing 0 jobs and 'workflow file issue' errors.
Both nuget.yml and deploy-server-side.yml now pass secrets through
env: block first, then reference env.VAR_NAME in the if condition.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@csharpfritz csharpfritz merged commit e4ae43e into FritzAndFriends:main Feb 25, 2026
4 checks passed
@csharpfritz csharpfritz deleted the ci-fix-promote branch February 25, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant