Skip to content

Align docs, manifest, and comments with current repo state#33

Open
shakersMSFT wants to merge 5 commits into
microsoft:mainfrom
shakersMSFT:update-developer-guide
Open

Align docs, manifest, and comments with current repo state#33
shakersMSFT wants to merge 5 commits into
microsoft:mainfrom
shakersMSFT:update-developer-guide

Conversation

@shakersMSFT
Copy link
Copy Markdown
Collaborator

@shakersMSFT shakersMSFT commented May 28, 2026

Summary

Housekeeping PR to align documentation, manifest paths, and code comments with the actual state of the repository.

Changes

Rename calm-oswindows-dev-config

  • Renamed src/tests/calm-os/ directory to src/tests/windows-dev-config/
  • Updated flow id in src/windows-dev-config/install.ps1 (-Id 'windows-dev-config')
  • Updated cmdpal README category examples

Polish development.md

  • Fixed title: "WindowsDevSetupScripts" → "WindowsDeveloperConfig"
  • Fixed DSC resource type names to match actual configs (RunCommandOnSet / PowerShellScript instead of Script / PSDscResources/Script)
  • Restructured repo layout tree to show src/ nesting clearly
  • Fixed table alignment and spelling

Remove stale references

  • Replaced mac-comfort-shell / mac-my-wsl comment references with current flow names
  • Removed orphaned src/tests/wsl-comfort-shell/ directory (unused; manifest uses comfort-shell)

- Rename calm-os -> windows-dev-config throughout (flow id, test dir,
  install shim, manifest, comments)
- Fix manifest install/configuration paths to use src/ prefix consistently
  (CI runs unsigned source, not signed top-level copies)
- Fix development.md: correct title, DSC resource type names, path
  references, repo layout tree, table alignment, spelling
- Remove stale mac-comfort-shell / mac-my-wsl references from comments
- Remove orphaned src/tests/wsl-comfort-shell/ directory
- Update cmdpal README category examples
Comment thread src/manifest.yml Outdated
install: Workloads/winforms/install.ps1
configuration: Workloads/winforms/configuration.winget
build: dotnet build src/tests/winforms/hello.csproj -c Release -v q --nologo -o tests/winforms/bin
install: src/Workloads/winforms/install.ps1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As this file will be used with the cmdpal ext, I would change the test harness To append in src.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe let John handle this adjustment via #31

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

are you suggesting that I remove all the src/ changes or that we let #31 merge first?

Comment thread src/docs/development.md Outdated
Comment thread src/docs/development.md Outdated
Comment thread src/future/cmdpal/README.md Outdated
Comment thread src/manifest.yml Outdated
Comment thread src/manifest.yml Outdated
@shakersMSFT shakersMSFT force-pushed the update-developer-guide branch from 14205c8 to b20e099 Compare May 29, 2026 21:56
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