Skip to content

DO NOT MERGE: verify #97 (self-replace 1.5.1) against the Windows self-update tests - #101

Closed
krassx wants to merge 3 commits into
mainfrom
tmp/verify-97
Closed

krassx wants to merge 3 commits into
mainfrom
tmp/verify-97

Conversation

@krassx

@krassx krassx commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Throwaway: PR #100's tests plus #97's Cargo.lock, to run the Windows self-update checks against the bumped dependency. Will be closed and its branch deleted.

dependabot Bot and others added 3 commits October 30, 2026 13:42
Bumps the cargo-minor-patch group with 1 update: [self-replace](https://github.com/mitsuhiko/self-replace).


Updates `self-replace` from 1.5.0 to 1.5.1
- [Changelog](https://github.com/mitsuhiko/self-replace/blob/main/CHANGELOG.md)
- [Commits](mitsuhiko/self-replace@1.5.0...1.5.1)

---
updated-dependencies:
- dependency-name: self-replace
  dependency-version: 1.5.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: cargo-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
The existing update test swaps in a sentinel on every OS. On Windows, self-replace is not a
rename (the running .exe is moved aside and a helper deletes the leftovers), so add a Windows-only
test that swaps in a real binary and checks the replaced .exe launches, that it can update again
over a file a self-replace put there, and that nothing is left in its directory afterwards.
Run it as a dedicated check on both Windows architectures so a bump of self-replace or
windows-sys (e.g. #97) fails by name.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WeiJxQWhjhQvvURAjiDkti
@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@krassx

krassx commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Verification done: PR #100's tests plus #97's lockfile pass on both Windows architectures (self-update, test, e2e, aarch64 build). Closing the throwaway.

@krassx krassx closed this Nov 3, 2026
@krassx
krassx deleted the tmp/verify-97 branch November 3, 2026 07:05
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.

1 participant