Skip to content

Update terminal size#19

Merged
sourcefrog merged 4 commits intomainfrom
update-terminal-size
Mar 7, 2026
Merged

Update terminal size#19
sourcefrog merged 4 commits intomainfrom
update-terminal-size

Conversation

@sourcefrog
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 7, 2026 15:32
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates terminal width detection to use the newer terminal_size API while keeping stdout/stderr width measurement platform-specific (Unix vs Windows).

Changes:

  • Bump terminal_size dependency from 0.2 to 0.4.
  • Refactor terminal-width logic into OS-specific modules and use terminal_size_of() with stdout/stderr handles/FDs.
  • Bump crate version to 0.1.6 and add a changelog entry.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/width.rs Routes width lookup through unix/windows modules behind appropriate cfg gates.
src/width/unix.rs Uses terminal_size_of(stdout().as_fd()) / stderr().as_fd() to measure widths on Unix.
src/width/windows.rs Uses terminal_size_of(stdout().as_handle()) / stderr().as_handle() to measure widths on Windows.
Cargo.toml Bumps crate version to 0.1.6 and updates terminal_size to 0.4.
NEWS.md Adds 0.1.6 release notes mentioning the dependency update.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sourcefrog sourcefrog merged commit f7c199f into main Mar 7, 2026
10 checks passed
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