Conversation
added 30 commits
August 16, 2025 08:04
## Summary - test self-hosted workflow for build-vi-package action ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` ------ https://chatgpt.com/codex/tasks/task_e_68a099e474f48329ad03bc6cb9559c8d
## Summary - add Pester test to validate missing-in-project action usage ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` ------ https://chatgpt.com/codex/tasks/task_e_68a09be1ad7483299120a53b30b7fbc5
## Summary - add Pester test ensuring close-labview workflow is exercised for 32- and 64-bit runs and uploads logs ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(fails: command not found)* ------ https://chatgpt.com/codex/tasks/task_e_68a09bdb3b148329994c69344a536f81
## Summary - add Pester test for modify-vipb-display-info self-hosted workflow ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Import-Module Pester -MinimumVersion 5.0.0; Invoke-Pester -CI -Path ./tests/pester"` ------ https://chatgpt.com/codex/tasks/task_e_68a09bea57a8832982068b8686d17d84
## Summary - add Pester test to verify `prepare-labview-source` workflow inputs and artifact upload ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(container failed: AddTokenToLabview.SelfHosted.Workflow.Tests.ps1, ApplyVipc.SelfHosted.Workflow.Tests.ps1, Build.SelfHosted.Workflow.Tests.ps1, BuildLvlibp.SelfHosted.Workflow.Tests.ps1, PrepareLabviewSource.SelfHosted.Workflow.Tests.ps1)* ------ https://chatgpt.com/codex/tasks/task_e_68a09beffd188329b72844b08b4c7b3b
## Summary - test rename-file action in workflows for self-hosted runner usage and artifact upload ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(fails: command not found)* ------ https://chatgpt.com/codex/tasks/task_e_68a09bf882d48329b3663faae36dc9b4
## Summary - test restore-setup-lv-source composite action usage in workflows ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(fails: command not found)* ------ https://chatgpt.com/codex/tasks/task_e_68a09bfde8288329b39632f13ecee99a
## Summary - add Pester test verifying self-hosted workflow calls revert-development-mode action and uploads config artifact ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` ------ https://chatgpt.com/codex/tasks/task_e_68a09c02962083298299efacb24e9fd2
## Summary - verify self-hosted workflow calls run-unit-tests action - ensure unit test results uploaded as artifact ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` ------ https://chatgpt.com/codex/tasks/task_e_68a09c07a1808329a6fb14d2f51b205a
## Summary - add workflow test ensuring set-development-mode action uses relative_path and flags ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Import-Module Pester; Invoke-Pester -CI -Path ./tests/pester"` ------ https://chatgpt.com/codex/tasks/task_e_68a0a3180ea48329af90f4976bd480d6
## Summary - add external GitHub Actions workflow that runs close-labview for 32-bit and 64-bit ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(fails: command not found)* ------ https://chatgpt.com/codex/tasks/task_e_68a0a6aa88dc83298bb2e65f0719e0a7
## Summary - clarify that REQ-012's workflow clones and operates on an external repository without modifying it ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` -⚠️ `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` (missing `pwsh`, attempted install failed) ------ https://chatgpt.com/codex/tasks/task_e_68a0a87a302c8329a5c3b46425b633a4
## Summary - ensure test targets `close-labview-external.yml` - check for 32- and 64-bit action runs - assert log artifacts are uploaded ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(fails: Container failed: 14)* ------ https://chatgpt.com/codex/tasks/task_e_68a0a9c9113c83298bde677bc1508980
## Summary - validate `close-labview` workflow jobs run only on self-hosted LabVIEW runners ## Testing - `npm run check:node` - `npm install` - `npm test` - `npx --yes markdownlint-cli README.md docs/**/*.md scripts/**/*.md` - `npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `pwsh -NoLogo -Command "Invoke-Pester -CI -Path ./tests/pester"` *(fails: command not found)* ------ https://chatgpt.com/codex/tasks/task_e_68a0afe5873c8329a3e64af1494b22f5
added 27 commits
August 17, 2025 21:16
## Summary - exercise apply-vipc composite action without cloning external repo - regenerate requirements documentation - update workflow and tests accordingly ## 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: 19 failed, 15 skipped)* - `pwsh -NoLogo -Command 'Invoke-Pester -Path "./tests/pester/ApplyVipc.Workflow.Tests.ps1"'` ------ https://chatgpt.com/codex/tasks/task_e_68a2a5fd25388329947f49b87f69d237
## Summary - remove external LabVIEW icon editor repository from self-hosted workflows - switch workflow paths to local fixtures and update Pester tests ## 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"` *(failed: New-PesterConfiguration not recognized)* ------ https://chatgpt.com/codex/tasks/task_e_68a2a961832c83299f4c309b58960b04
## Summary - Allow default dispatcher args when labview-icon-editor repo is absent - Add RequireProject switch and LABVIEW_ICON_EDITOR_REQUIRED env var for strict checks - Document optional labview-icon-editor usage in Pester guide ## Testing - `apt-get update && apt-get install -y apt-utils` - `PATH=/usr/bin:$PATH npm run check:node` - `PATH=/usr/bin:$PATH npm install` - `PATH=/usr/bin:$PATH npm test` - `PATH=/usr/bin:$PATH npm run lint:md` - `PATH=/usr/bin:$PATH npx --yes markdown-link-check -q -c .markdown-link-check.json README.md $(find docs scripts -name '*.md')` - `/root/.local/share/mise/installs/go/1.24.3/bin/actionlint` - `pwsh -NoLogo -Command "$cfg = New-PesterConfiguration; $cfg.Run.Path = './tests/pester'; $cfg.TestResult.Enabled = $false; Invoke-Pester -Configuration $cfg"` *(fails: ConvertFrom-Yaml cmdlet missing)* ------ https://chatgpt.com/codex/tasks/task_e_68a2ad24673c8329bd0ba2ec785b9221
## Summary - manually install PowerShell 7.5.1 on Windows runners via MSI and checksum validation - remove `actions/setup-pwsh` from CI ## 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: Expected 0, but got 1)* - `gh workflow run ci.yml` *(fails: command not found)* ------ https://chatgpt.com/codex/tasks/task_e_68a2b2938e1c8329ae77805858251dec # Conflicts: # .github/workflows/ci.yml
## Summary - streamline Invoke-OSAction argument parsing to JSON or hashtable inputs ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required)* - `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: multiple tests due to missing ConvertFrom-Yaml and failing dry run expectations)* ------ https://chatgpt.com/codex/tasks/task_e_68a2bd146e448329a97ad4d200ca6ea5
## Summary - convert workflow YAML files to JSON and update Pester tests to parse JSON - document JSON-based dispatcher usage and remove ConvertFrom-Yaml references ## Testing - `npm run check:node` - `npm install` - `npm test` - `npm run lint:md` - `npx --yes markdown-link-check -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: Expected 0, but got 1 in Dispatcher.DryRun tests)* ------ https://chatgpt.com/codex/tasks/task_e_68a2bfb772448329ba7f34f8087f876d
## Summary - clarify in README that ConvertFrom-Yaml is no longer required - show JSON ArgsFile examples for run-unit-tests, quickstart, and dispatcher docs - update common parameters to use ArgsJson instead of ArgsYaml ## 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: Tests Passed: 89, Failed: 16, Skipped: 15, Inconclusive: 0, NotRun: 0)* ------ https://chatgpt.com/codex/tasks/task_e_68a2c32408548329a659a7919d4e9171
## Summary - pass dispatch arguments as JSON in all composite actions - update Pester workflows to supply explicit JSON config - adjust documentation generator for JSON examples ## Testing - `npm run check:node` - `npm install` - `npm test` - `npm run lint:md` - `npx --yes markdown-link-check -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: Tests Passed: 89, Failed: 16)* ------ https://chatgpt.com/codex/tasks/task_e_68a2c55bbb8c83298c8678d6fe22eb72
## Summary - switch dispatcher override to ArgsJson to match JSON argument semantics ## 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: Tests Passed: 89, Failed: 16)* - `pwsh -NoLogo -Command '$cfg = New-PesterConfiguration; $cfg.Run.Path = "./tests/pester/Dispatcher.Tests.ps1"; $cfg.TestResult.Enabled = $false; Invoke-Pester -Configuration $cfg'` ------ https://chatgpt.com/codex/tasks/task_e_68a2c7d85c1883299209b4ec6036aab3
## Summary - remove legacy ConvertFrom-Yaml instructions in favor of JSON args files ## Testing - `npm run check:node` *(fails: Node.js >=24 required, but 20.19.4 found)* - `npm install` - `npm test` *(fails: Node.js >=24 required, but 20.19.4 found)* - `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: Tests Passed: 89, Failed: 16, Skipped: 15)* ------ https://chatgpt.com/codex/tasks/task_e_68a2cc088b688329a9906cabd4f3c983
Signed-off-by: sergiov-ni <sergio.velderrain@ni.com>
## Summary - rename module functions to use approved verbs with hyphenated names - update dispatcher registry, exports, and docs - adjust tests for new function names ## 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: Tests Passed: 88, Failed: 17, Skipped: 15)* ------ https://chatgpt.com/codex/tasks/task_e_68a2d30cb1c88329818118e3d8cd5c1f
## Summary - remove strict OS build check when installing PowerShell 7.5.1 ## 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: 88 passed, 17 failed, 15 skipped)* ------ https://chatgpt.com/codex/tasks/task_e_68a2d7fc13ec8329b41c95112c9a3c24
## Summary - run Pester with explicit configuration and disabled XML output - document that CI's Pester run skips XML reports - note XML test output is disabled across docs ## 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"` ------ https://chatgpt.com/codex/tasks/task_e_68a2133370c88329a3bb0a3e558b9e0a
svelderrainruiz
pushed a commit
that referenced
this pull request
Aug 18, 2025
svelderrainruiz
pushed a commit
that referenced
this pull request
Aug 19, 2025
…statuscounts-helper-function feat: centralize test status counting
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.