Skip to content

test(land): skip the fixer suites on Windows, like the daemon's - #315

Merged
moq-bot[bot] merged 2 commits into
mainfrom
fix/windows-skip-fixer
Oct 31, 2026
Merged

moq-bot[bot] merged 2 commits into
mainfrom
fix/windows-skip-fixer

Conversation

@kixelated

Copy link
Copy Markdown
Collaborator

main's native Windows jobs have failed on every fixer test since #303: the "asynchronous land fixer" suite and test/land-fixer-diagnostics.test.ts use fifos, POSIX shells, and Linux process groups. The daemon suites beside them already skip on Windows. These are optional checks, so it went unnoticed.

Both now skip on win32, like the daemon's.

Checks

just check passes. The land and diagnostics tests pass on Linux (118).

Review

Skipped: two skipIf guards matching the neighbouring daemon suites.

🤖 Generated with Claude Code

(written by Claude Opus 5.5)

main's native Windows jobs have failed on every fixer test since #303: they use
fifos, POSIX shells, and Linux process groups, as the daemon does.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d42d1ecd-f4bb-440b-8727-fb0da52432d4
  • 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

@kixelated
kixelated enabled auto-merge (squash) October 31, 2026 06:29
@moq-bot
moq-bot Bot merged commit ab0f836 into main Oct 31, 2026
@moq-bot
moq-bot Bot deleted the fix/windows-skip-fixer branch October 31, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant