Skip to content

Fix stale StoreApp references from pre-eShopLite Part 9 layout - #597

Merged
jongalloway merged 1 commit into
dotnet-presentations:mainfrom
ShiyunXu:fix/storeapp-references
Aug 26, 2026
Merged

Fix stale StoreApp references from pre-eShopLite Part 9 layout#597
jongalloway merged 1 commit into
dotnet-presentations:mainfrom
ShiyunXu:fix/storeapp-references

Conversation

@ShiyunXu

Copy link
Copy Markdown
Contributor

Part 9 is now the eShopLite/ + eShopLite-start/ Aspire solutions, but three files still referenced the removed StoreApp/ console snapshot:

  • .github/dependabot.yml — its NuGet directories list pointed at /Part 09 - Adding AI to an Existing App/StoreApp/, which no longer exists. Dependabot fails on that entry and Part 9's ten projects (two 5-project solutions) get no update coverage. Replaced with the eShopLite-start/ and eShopLite/ directories.
  • .github/skills/workshop-testing/SKILL.md — the part-by-part table told testers to "run the completed StoreApp snapshot". Now describes the actual flow (work in eShopLite-start/, reconcile against the eShopLite/ answer key, optional LocalModel:* secrets in the Store project).
  • .github/skills/workshop-testing/report-template.md — snapshot reconciliation table now lists both Part 9 solutions.

The July 2026 test report (docs/testing/workshop-test-report-2026-07-27.md) flagged the SKILL.md issue; this resolves it. Markdownlint passes on both edited markdown files.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Part 9 is now the eShopLite / eShopLite-start Aspire solutions, but
three files still pointed at the removed StoreApp console snapshot:

- dependabot.yml pointed its NuGet update coverage at a directory that
  no longer exists, leaving Part 9's ten projects without updates
- SKILL.md's part table told testers to run the StoreApp snapshot
- report-template.md's snapshot reconciliation table listed StoreApp

The July 2026 test report (docs/testing/workshop-test-report-2026-07-27.md)
flagged the SKILL.md issue; this resolves it.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jongalloway
jongalloway merged commit 6739196 into dotnet-presentations:main Aug 26, 2026
2 of 3 checks passed
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.

3 participants