Skip to content

Commit 62c9900

Browse files
WilliamBerryiiiBill Berry
andauthored
fix(build): resolve CI failures for release 0.5.0 PR (#174)
## Description Resolved two CI failures blocking the release-please 0.5.0 PR (#173): **Spell Check** and **Markdown Link Check**. The Spell Check failure was caused by three unrecognized terms in the new 0.5.0 CHANGELOG section — *MLflow*, *Dependabot*, and *CSRF* — that were not present in the cspell custom dictionaries. Added each term to the appropriate dictionary file. The Markdown Link Check failure was caused by legacy repository URLs in *CHANGELOG.md* pointing to the archived `Azure-Samples/azure-nvidia-robotics-reference-architecture` location. Since all tags and history were migrated to `microsoft/physical-ai-toolchain`, replaced all 99 legacy URLs with the current repository path. The updated URLs match the existing link checker `ignorePattern`, resolving the CI failure. > These fixes are intentionally minimal and surgical — only the specific CI blockers were addressed, with no refactoring or restructuring of existing content. Related to PR #173 ## Type of Change - [x] 🐛 Bug fix (non-breaking change fixing an issue) - [ ] ✨ New feature (non-breaking change adding functionality) - [ ] 💥 Breaking change (fix or feature causing existing functionality to change) - [ ] 📚 Documentation update - [ ] 🏗️ Infrastructure change (Terraform/IaC) - [ ] ♻️ Refactoring (no functional changes) ## Component(s) Affected - [ ] `deploy/000-prerequisites` - Azure subscription setup - [ ] `deploy/001-iac` - Terraform infrastructure - [ ] `deploy/002-setup` - OSMO control plane / Helm - [ ] `deploy/004-workflow` - Training workflows - [ ] `src/training` - Python training scripts - [ ] `docs/` - Documentation ## Testing Performed - [ ] Terraform `plan` reviewed (no unexpected changes) - [ ] Terraform `apply` tested in dev environment - [ ] Training scripts tested locally with Isaac Sim - [ ] OSMO workflow submitted successfully - [ ] Smoke tests passed (`smoke_test_azure.py`) **Additional validation performed:** - `npm run spell-check` — 0 issues across 399 files after dictionary additions - `npx markdown-link-check CHANGELOG.md` — all replaced URLs confirmed IGNORED by existing pattern - Tags v0.1.0–v0.5.0 verified present on `microsoft/physical-ai-toolchain` with HTTP 200 responses ## Documentation Impact - [x] No documentation changes needed - [ ] Documentation updated in this PR - [ ] Documentation issue filed ## Bug Fix Checklist *Complete this section for bug fix PRs. Skip for other contribution types.* - [x] Linked to issue being fixed - [ ] Regression test included, OR - [x] Justification for no regression test: CI checks (Spell Check, Markdown Link Check) themselves serve as the regression mechanism — the same CI pipeline that detected these failures will validate the fix on merge. ## Checklist - [x] My code follows the [project conventions](copilot-instructions.md) - [x] Commit messages follow [conventional commit format](instructions/commit-message.instructions.md) - [x] I have performed a self-review Signed-off-by: Bill Berry <wbery@microsoft.com> Co-authored-by: Bill Berry <wbery@microsoft.com>
1 parent baf9e58 commit 62c9900

3 files changed

Lines changed: 102 additions & 99 deletions

File tree

.cspell/azure-services.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,14 @@ azuremonitor
4040
azureml
4141
containerapp
4242
cosmosdb
43+
Dependabot
4344
deviceregistry
4445
eastus
4546
Entra
4647
entra
4748
Eventhouse
4849
keyvault
50+
MLflow
4951
onedrive
5052
onenote
5153
powerbi

.cspell/industry-acronyms.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ BYOK
77
CAIRA
88
CCPA
99
CLDR
10+
CSRF
1011
CMMS
1112
COMMITMSG
1213
C-SCRM

0 commit comments

Comments
 (0)