Skip to content

Update codegen files - #56

Merged
github-actions[bot] merged 1 commit into
mainfrom
codegen
Apr 16, 2026
Merged

Update codegen files#56
github-actions[bot] merged 1 commit into
mainfrom
codegen

Conversation

@ptr727-codegen

Copy link
Copy Markdown
Contributor

This PR updates the codegen files.

@github-actions
github-actions Bot enabled auto-merge (squash) April 16, 2026 20:37
@github-actions
github-actions Bot merged commit dfeda34 into main Apr 16, 2026
20 checks passed
ptr727 added a commit that referenced this pull request Aug 17, 2026
## What

Adds `powershell` to the host tools `install-tools.sh` manages, as the
one optional member. It installs from Microsoft's apt repository, the
same feed `dotnet` already falls back to, so no separate download step
is needed (the path HomeAutomation-Config's PR #56 took with a tar.gz
build).

- `install-tools.sh --optional` (or a stand-up's `--optional` tools
step) now also installs `pwsh`.
- Naming it explicitly (`install-tools.sh --install powershell`) selects
it regardless of `--optional`.
- The feed registration is extracted into a shared `microsoft_feed()`,
used by both `dotnet` and `powershell`; `dotnet`'s behavior is
unchanged.
- `--list` marks it `(optional)`, and the report notes when Microsoft's
repository is not configured.
- Non-amd64 hosts get a named skip, matching `dotnet`.

Verified: shellcheck, prose_lint, markdownlint, and editorconfig are
clean, and all 686 tests in `scripts/tests` pass. On a Debian 13 host
the report shows `powershell 7.6.5-1.deb` available from
`packages.microsoft.com`, and a default report (without `--optional`)
omits it.

## Docs

- `host-setup/linux/README.md` source taxonomy and the
docker/node/dotnet section.
- `docs/host-setup.md` source paragraph and the "not in the table"
sentence.
ptr727 added a commit that referenced this pull request Aug 17, 2026
… Pilot (#778)

## What

- `.github/actions/build-executable-default/action.yml`: the release
archive is named for the project file's stem
(`./PhotoCleaner/PhotoCleaner.csproj` gives `PhotoCleaner.7z`) unless
the new `asset-name` input names it, and `build-release-task.yml`
exposes that as `executable_asset_name`. PhotoCleaner's first publish
through the task (run 31977092102, release 1.1.11) attached `Console.7z`
where its own leaf had attached `PhotoCleaner.7z`, the one regression
the pilot found. PlexCleaner names its archive the same way, so the
derived default is right for both known callers.
- `catalog/snippets/workflows/publish-release.yml` and the doc's
byte-identical stub: `permissions: {}` at workflow level, `contents:
read` on the `validate` job, the publish job's grants unchanged. These
are the four suppressed findings raised on #775 after its clean round.
- `docs/reusable-workflows.md` Rollout: PhotoCleaner ticked as the stage
2 pilot (ptr727/PhotoCleaner#55, #56, the validate default-hook path
observed) and the stage 4 pilot (smoke run 31974932749 and publish run
31977092102, release 1.1.11), a proof item for the asset-name fix on the
next PhotoCleaner release.
- `reports/workflow-reuse.md` regenerated: PhotoCleaner 8 workflow files
to 3, all three callers, fleet callers 1 to 3.

## Verification

actionlint, prose gate, repo gate, audit self-test, unit tests,
markdownlint. The asset-name change is proven by the next PhotoCleaner
release, recorded as a proof item.

Refs ptr727/PhotoCleaner#55, ptr727/PhotoCleaner#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.

0 participants