Skip to content

Remove unused environment variables from maintenance workflow#53

Closed
Copilot wants to merge 2 commits intoupdate-v9from
copilot/sub-pr-49-yet-again
Closed

Remove unused environment variables from maintenance workflow#53
Copilot wants to merge 2 commits intoupdate-v9from
copilot/sub-pr-49-yet-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 16, 2025

The maintenance workflow was setting HF_TEST and CIVITAI_TEST environment variables immediately before exit 1. Environment variables don't propagate between GitHub Actions jobs, making these assignments useless for the downstream copilot-auto-fix job.

Changes

  • Removed HF_TEST and CIVITAI_TEST environment variable assignments from the "Evaluate results" step
  • Simplified step to just exit 1 when tests fail

The test outcomes are already captured via step IDs (steps.hf_test.outcome and steps.civitai_test.outcome) and accessible through the job's conditional logic if needed.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: suzukimain <131413573+suzukimain@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on model loading enhancement Remove unused environment variables from maintenance workflow Dec 16, 2025
Copilot AI requested a review from suzukimain December 16, 2025 10:10
@suzukimain suzukimain closed this Dec 16, 2025
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.

2 participants