Skip to content

📖 [Docs]: Module-local workflow phases documented as root-only#100

Merged
Marius Storhaug (MariusStorhaug) merged 1 commit into
mainfrom
clarify-test-workflow-phases
Jul 26, 2026
Merged

📖 [Docs]: Module-local workflow phases documented as root-only#100
Marius Storhaug (MariusStorhaug) merged 1 commit into
mainfrom
clarify-test-workflow-phases

Conversation

@MariusStorhaug

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Jul 26, 2026

Copy link
Copy Markdown
Member

Module authors can now distinguish the two root-only workflow phase scripts from recursively discovered module-local test entries.

Fixed: Setup and teardown discovery is unambiguous

tests/BeforeAll.ps1 and tests/AfterAll.ps1 are documented as exact, non-recursive paths that run once around the module-local test matrix. Ordinary Advanced test entries remain separately eligible for recursive discovery under tests/.


Technical details
  • Corrects src/docs/Frameworks/Process-PSModule/pipeline-stages.md to match the exact Test-Path checks in Process-PSModule main.
  • Separates workflow phase detection from recursive configuration, container, and test-file discovery.
  • Implementation plan progress: all tasks in Clarify module-local setup and teardown discovery #99 are complete.
  • Validation passed: documentation indexes, links, Zensical build, Super-Linter, and diff checks.
  • Copilot review converged after the clarity feedback was addressed; the final round added no comments.
Relevant issues (or links)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Process-PSModule pipeline documentation to clearly distinguish root-only workflow phase scripts (tests/BeforeAll.ps1 / tests/AfterAll.ps1) from recursively discovered module-local tests under tests/, aligning the docs with the current workflow detection behavior.

Changes:

  • Documented tests/BeforeAll.ps1 and tests/AfterAll.ps1 as exact, repository-root-only (non-recursive) workflow phase paths.
  • Clarified that nested BeforeAll.ps1 / AfterAll.ps1 files under tests/ do not create workflow phases.

Comment thread src/docs/Frameworks/Process-PSModule/pipeline-stages.md Outdated
Comment thread src/docs/Frameworks/Process-PSModule/pipeline-stages.md Outdated
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

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

@MariusStorhaug
Marius Storhaug (MariusStorhaug) marked this pull request as ready for review July 26, 2026 07:18
@MariusStorhaug
Marius Storhaug (MariusStorhaug) merged commit 65591bd into main Jul 26, 2026
20 checks passed
@MariusStorhaug
Marius Storhaug (MariusStorhaug) deleted the clarify-test-workflow-phases branch July 26, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docs NoRelease No release required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify module-local setup and teardown discovery

2 participants