Skip to content

Correct Windows guidance on system settings - #3372

Merged
max-sixty merged 3 commits into
max-sixty:mainfrom
ofek:patch-1
Jul 7, 2026
Merged

Correct Windows guidance on system settings#3372
max-sixty merged 3 commits into
max-sixty:mainfrom
ofek:patch-1

Conversation

@ofek

@ofek ofek commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

On this page describing a Windows update you can find the following text if you expand the Settings drop-down section.

New! This update changes the design of a few lists to be more consistent with other Settings pages. The lists are for Settings > Apps > Startup Apps and Settings > Apps > Advanced App Settings > App Execution Alias. The new design also makes it easier to view more information about the apps you have.

Here is a screenshot taken from my machine:

image

@worktrunk-bot worktrunk-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The corrected navigation path is accurate, and README.md + docs/content/worktrunk.md are updated consistently. One thing is missing though: there's a third generated mirror of this text that still has the old path, which is why test_docs_are_in_sync is failing.

skills/worktrunk/reference/worktrunk.md is regenerated from docs/content/worktrunk.md, and its line still reads:

Alternatively, disable Windows Terminal's alias (Settings → Privacy & security → For developers → App Execution Aliases → disable "Windows Terminal") to use `wt` directly.

It needs the same edit the other two files got. The failure currently shows up on affected tests (linux, advisory), but the same test_docs_are_in_sync test also runs in the required test (linux/macos/windows) jobs, so it will block the merge once those finish.

The fix is mechanical — run cargo test --test integration test_docs_are_in_sync locally (it rewrites the mirror in place) and commit the result, or just apply the identical one-line change to skills/worktrunk/reference/worktrunk.md. Happy to push that commit to this branch if you'd like — just say the word.

The third generated mirror still had the old App Execution Aliases
navigation path; test_docs_are_in_sync flagged it. Apply the same
one-line edit as README.md and docs/content/worktrunk.md.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@max-sixty
max-sixty merged commit 93edf90 into max-sixty:main Jul 7, 2026
38 checks passed
@ofek
ofek deleted the patch-1 branch July 7, 2026 07:03
max-sixty added a commit that referenced this pull request Jul 8, 2026
…indow (#3386)

Five commits landed on `main` during #3385's CI wait, so they ship
inside v0.66.0's tree but weren't in its changelog (the step-12 drift
check in the release skill caught this). This adds the user-facing three
to the 0.66.0 section:

- **Codex activity markers**
([#3364](#3364), closes
[#3362](#3362), thanks
@ofek for reporting)
- **SCP-style SSH remotes with custom usernames**
([#3371](#3371), thanks
@fcoury-oai)
- **Windows app-alias Settings path**
([#3372](#3372), thanks
@ofek)

Excluded as internal: #3360 (trait docstrings), #3379 (argv[0] display
refactor). Every entry verified against the actual diffs.

The v0.66.0 tag will point at this PR's squash commit so the tagged tree
carries the complete changelog.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

> _This was written by Claude Code on behalf of Maximilian Roos_

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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