Skip to content

Conversation

@PakhomovAlexander
Copy link
Contributor

Summary

  • Disable pager by default on Windows because the more command doesn't support ANSI color codes well
  • Other platforms (Linux, macOS) still have pager enabled by default using less -RFX
  • Users can explicitly configure via cli config set ignite.cli.pager.enabled=true/false

Jira: https://issues.apache.org/jira/browse/IGNITE-27620

Test plan

  • PagerSupportTest passes
  • Checkstyle/PMD clean

🤖 Generated with Claude Code

The Windows 'more' command doesn't support ANSI color codes well,
causing garbled output when paging through colored CLI output.

Change the default pager behavior:
- Windows: disabled by default
- Other platforms: enabled by default (uses 'less -RFX')

Users can still explicitly enable/disable pager via config:
  cli config set ignite.cli.pager.enabled=true

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@PakhomovAlexander PakhomovAlexander marked this pull request as draft January 23, 2026 12:00
PakhomovAlexander and others added 2 commits January 23, 2026 18:41
Tests verify that:
- Pager is disabled by default on Windows
- Pager is enabled by default on Linux/macOS
- Explicit config overrides the default

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Replace system property manipulation with JUnit 5 OS condition
annotations for cleaner, more idiomatic tests.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@PakhomovAlexander PakhomovAlexander marked this pull request as ready for review January 23, 2026 16:12
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