Skip to content

Improve word wrapping #327

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 21, 2025

Conversation

roulpriya
Copy link
Contributor

Issue #251

Implemented automatic text wrapping functionality to ensure Swiftly's terminal output respects the current terminal width.

Changes

  • Added wrapText() extension to String that reformats text to fit within a specified column width
  • Integrated terminal width detection in the output system using ioctl and winsize

@cmcgee1024
Copy link
Member

@swift-ci test macOS

@roulpriya roulpriya force-pushed the improve-word-wrap-for-terminal branch from 39458ad to 939cfae Compare April 21, 2025 11:45
@roulpriya
Copy link
Contributor Author

There was a swift formatting error. I fixed it.
Can you retrigger the CI

@cmcgee1024
Copy link
Member

@swift-ci test macOS

@roulpriya roulpriya force-pushed the improve-word-wrap-for-terminal branch from 939cfae to 99860c4 Compare April 21, 2025 14:41
@cmcgee1024
Copy link
Member

@swift-ci test macOS

@roulpriya roulpriya force-pushed the improve-word-wrap-for-terminal branch from 99860c4 to 625618e Compare April 21, 2025 15:06
@roulpriya
Copy link
Contributor Author

I fixed the CI failure; please retrigger(Tested in Ubuntu)

@cmcgee1024
Copy link
Member

@swift-ci test macOS

@cmcgee1024
Copy link
Member

Thanks @roulpriya for your contribution. This looks good.

@cmcgee1024 cmcgee1024 merged commit 4fca65e into swiftlang:main Apr 21, 2025
23 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.

2 participants