Skip to content

Comments

chore: use plural test result env var#2

Merged
svelderrainruiz merged 1 commit intocodex/update-ci-workflow-for-pester-resultsfrom
codex/update-test_results_glob-to-test_results_globs
Aug 18, 2025
Merged

chore: use plural test result env var#2
svelderrainruiz merged 1 commit intocodex/update-ci-workflow-for-pester-resultsfrom
codex/update-test_results_glob-to-test_results_globs

Conversation

@svelderrainruiz
Copy link

Summary

  • use TEST_RESULTS_GLOBS in CI workflows
  • adjust tests to expect plural env var

Testing

  • npm run check:node
  • npm install
  • npm test
  • npm run lint:md
  • npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')
  • actionlint
  • pwsh -NoLogo -Command "$cfg = New-PesterConfiguration; $cfg.Run.Path = './tests/pester'; $cfg.TestResult.Enabled = $false; Invoke-Pester -Configuration $cfg" (fails: 17 tests)

https://chatgpt.com/codex/tasks/task_e_68a37dde221883299166edba3ff02afa

@svelderrainruiz svelderrainruiz merged commit d22bbd6 into codex/update-ci-workflow-for-pester-results Aug 18, 2025
@svelderrainruiz svelderrainruiz deleted the codex/update-test_results_glob-to-test_results_globs branch August 18, 2025 19:32
svelderrainruiz pushed a commit that referenced this pull request Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant