Skip to content

Conversation

@MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Jan 12, 2026

This patch release makes test secrets available in the BeforeAll-ModuleLocal and AfterAll-ModuleLocal workflow jobs, enabling proper test setup and teardown operations that require authentication credentials.

Changes

Copilot AI review requested due to automatic review settings January 12, 2026 22:19
@MariusStorhaug MariusStorhaug requested a review from a team as a code owner January 12, 2026 22:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enables test-related secrets to be accessible in the BeforeAll and AfterAll workflow jobs by explicitly passing them from the main workflow to the reusable workflows and mapping them to environment variables.

Changes:

  • Added explicit secret passing to BeforeAll-ModuleLocal and AfterAll-ModuleLocal jobs in the main workflow
  • Declared secrets as inputs in both reusable workflow files with appropriate descriptions
  • Mapped secrets to environment variables in both reusable workflows

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/workflow.yml Added explicit secret passing to BeforeAll-ModuleLocal and AfterAll-ModuleLocal jobs (7 test-related secrets each)
.github/workflows/BeforeAll-ModuleLocal.yml Declared secrets as workflow inputs and mapped them to environment variables for test setup scripts
.github/workflows/AfterAll-ModuleLocal.yml Declared secrets as workflow inputs and mapped them to environment variables for test teardown scripts

@MariusStorhaug MariusStorhaug merged commit 9bf1a23 into main Jan 12, 2026
65 of 67 checks passed
@MariusStorhaug MariusStorhaug deleted the secrets-in-setup-teardown branch January 12, 2026 22:56
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