Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Preselect setup steps with pending managed config updates#38

Merged
ChangeHow merged 2 commits into
main from
copilot/fix-curl-incremental-update
May 16, 2026
Merged

Preselect setup steps with pending managed config updates #38
ChangeHow merged 2 commits into
main from
copilot/fix-curl-incremental-update

Conversation

Copilot AI commented May 16, 2026

Copy link
Copy Markdown
Contributor

Rerunning suitup via curl skipped already-completed setup steps, so new suitup-managed zsh config additions—such as deferred completion loading—were not offered for incremental merge.

  • Pending update detection

    • Added detection for suitup-managed files whose shipped config has safe line additions.
    • Covers .zshrc, .zshenv, core/shared zsh configs, tool modules, plugins, and aliases.
  • Setup step selection

    • Completed steps remain deselected only when no managed config update is pending.
    • Steps with pending additions are preselected so reruns can show the existing diff preview and confirmation flow.
  • Regression coverage

    • Added coverage for a completed zsh-config install with outdated completion.zsh, ensuring setup reruns select it for update.

Copilot AI and others added 2 commits May 15, 2026 18:14
Agent-Logs-Url: https://github.com/ChangeHow/suitup/sessions/8158088a-d88a-4bf7-af4b-f4d6f01474c8
Co-authored-by: ChangeHow <23733347+ChangeHow@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ChangeHow/suitup/sessions/8158088a-d88a-4bf7-af4b-f4d6f01474c8
Co-authored-by: ChangeHow <23733347+ChangeHow@users.noreply.github.com>
@ChangeHow ChangeHow marked this pull request as ready for review May 16, 2026 18:31
@ChangeHow ChangeHow merged commit 90c0916 into main May 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /